RE: [PATCH v7] USB: serial: cp210x: Add support for GPIOs on CP2108

[Pho Tran]

Hi Greg!
I am grateful for your promptly reply!
 Yesterday, I got the response from the kernel test robot with this message: 
" If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>". 
It's a reason why I added "Reported-by: kernel test robot <lkp@xxxxxxxxx>" 
to my new Patch.
Could you tell me what I need to do in the next step to submit this patch to the kernel mainline?
Once again, Thank you a lot!

-----Original Message-----
From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> 
Sent: Tuesday, April 6, 2021 12:15 PM
To: Pho Tran <photranvan0712@xxxxxxxxx>
Cc: johan@xxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Hung Nguyen <Hung.Nguyen@xxxxxxxxxx>; Tung Pham <Tung.Pham@xxxxxxxxxx>; Pho Tran <Pho.Tran@xxxxxxxxxx>; Kernel test robot <lkp@xxxxxxxxx>
Subject: Re: [PATCH v7] USB: serial: cp210x: Add support for GPIOs on CP2108

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.


On Tue, Apr 06, 2021 at 11:02:38AM +0700, Pho Tran wrote:
> From: Pho Tran <pho.tran@xxxxxxxxxx>
>
> Similar to other CP210x devices, GPIO interfaces (gpiochip) should be 
> supported for CP2108.
>
> CP2108 has 4 serial interfaces but only 1 set of GPIO pins are shared 
> to all of those interfaces. So, just need to initialize GPIOs of 
> CP2108 with only one interface (I use interface 0). It means just only 
> 1 gpiochip device file will be created for CP2108.
>
> CP2108 has 16 GPIOs, So data types of several variables need to be is 
> u16 instead of u8(in struct cp210x_serial_private). This doesn't 
> affect other CP210x devices.
>
> Because CP2108 has 16 GPIO pins, the parameter passed by cp210x 
> functions will be different from other CP210x devices. So need to 
> check part number of the device to use correct data format  before 
> sending commands to devices.
>
> Like CP2104, CP2108 have GPIO pins with configurable options. 
> Therefore, should be mask all pins which are not in GPIO mode in 
> cp2108_gpio_init() function.
>
> Fix build test WARNING reported by kernel test robot.
>
> Reported-by: Kernel test robot <lkp@xxxxxxxxx>

The kernel test robot did not report this needed support :(