Hi, On Tue, Jul 14, 2020 at 12:08:51PM +0200, Johan Hovold wrote: > On Fri, Jul 10, 2020 at 12:21:26AM +0200, Angelo Dureghello wrote: > > From some researches, this driver is available from the IC > > constructor site, but for older kernel versions. From there, decided > > to add a much simplier mainline version, written from scratch. > > Do you have a pointer to the vendor sources for reference? > > > This initial simple version is implemented without any flow control, > > tested mainly at 115200, but all standard baud rates are supported > > and applied as per serial terminal settings. > > > > Signed-off-by: Angelo Dureghello <angelo.dureghello@xxxxxxxxxxx> Just curious, how are you accessing this chip? I mean any breakout board or integrated in any custom board design. > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > > No need for a reported-by if the robot catches issues during review. > Just mention it in the changelog as you already did below. > > > --- > > Changes for v2: > > - fix test robot warning, PARITY_ defines renamed > > --- > > drivers/usb/serial/Kconfig | 9 + > > drivers/usb/serial/Makefile | 1 + > > drivers/usb/serial/xr21v1412.c | 361 +++++++++++++++++++++++++++++++++ > > 3 files changed, 371 insertions(+) > > create mode 100644 drivers/usb/serial/xr21v1412.c > > Thanks for the submitting this. > > There was another driver for this device posted recently, and which > appears to have more features: > > https://lore.kernel.org/r/20200607162350.21297-2-mani@xxxxxxxxxx > > Would you mind taking a look and see if that one would work for you? > > I had some comments on the latest version that needs to be addressed, > but I'm assuming Manivannan is working on a v5? > Yes! This driver is being worked on in my limited spare time. So I'm a bit late to reiterate the patchset but planning to send v5 around this weekend. Thanks, Mani > Johan