linux-kernel - Re: [PATCH v4] USB: serial: cp210x: Make the CP210x driver work with GPIOs of CP2108

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <YEsoE+QXSq8LagWG@hovoldconsulting.com>
Date:   Fri, 12 Mar 2021 09:36:35 +0100
From:   Johan Hovold <johan@...nel.org>
To:     Pho Tran <Pho.Tran@...abs.com>
Cc:     "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        Hung Nguyen <Hung.Nguyen@...abs.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>
Subject: Re: [PATCH v4] USB: serial: cp210x: Make the CP210x driver work with
 GPIOs  of CP2108

On Fri, Mar 12, 2021 at 04:27:57AM +0000, Pho Tran wrote:
> 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.
> 
> Signed-off-by: Pho Tran <pho.tran@...abs.com<mailto:pho.tran@...abs.com>>
> —
> 03/05/2021: Patch v3 modified format and contents of changelog follow feedback
> from Jonhan Hovold <johan@...nel.org<mailto:johan@...nel.org>>.
> 03/04/2021: Patch v2 modified format patch as comment from
> Johan Hovold <johan@...nel.org<mailto:johan@...nel.org>>:
> 1. Break commit message lines at 80 cols
> 2. Use kernel u8 and u16 types instead of the c99 ones.
> 03/01/2021: Initialed submission of patch “Make the CP210x driver work with
> GPIOs of CP2108.”.

Why are you resending the v4 that you submitted only four days ago? 

Note that this version is again white space corrupted, and something
happened to you SOB tag above.

Johan