linux-kernel - Re: [PATCH] parport_serial: Add support for WCH CH382L PCI-E single parallel port card.

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20170618213748.28e312c6@brick.gerasiov.net>
Date:   Sun, 18 Jun 2017 21:37:48 +0300
From:   Alexander Gerasiov <gq@...lab-i.ru>
To:     Andy Shevchenko <andy.shevchenko@...il.com>
Cc:     Sudip Mukherjee <sudipm.mukherjee@...il.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] parport_serial: Add support for WCH CH382L PCI-E single
 parallel port card.

Hello Andy,

While preparing the update I suddenly found, that parport_serial.c is
not the right place for this card. Since there is no serial ports on
the board, it would be better to put it into parport_pc.c

So I'll resubmit the patch right after I get access to the hardware to
check, that I did it the right way.

On Sat, 17 Jun 2017 16:45:49 +0300
Andy Shevchenko <andy.shevchenko@...il.com> wrote:

> On Sat, Jun 17, 2017 at 4:02 PM, Sudip Mukherjee
> <sudipm.mukherjee@...il.com> wrote:
> > From: Alexander Gerasiov <gq@...lab-i.ru>
> >
> > WCH CH382L is a PCI-E adapter with 1 parallel port. It is similair
> > to CH382 but serial ports are not soldered on board. Detected as
> > Serial controller: Device 1c00:3050 (rev 10) (prog-if 05 [16850])  
> 
> >         wch_ch382_2s1p,
> > +       wch_ch382_0s1p,  
> 
> >         /* wch_ch382_2s1p*/             { 1, { { 2, -1}, } },
> > +       /* wch_ch382_0s1p*/             { 1, { { 2, -1}, } },  
> 
> >         { 0x1c00, 0x3250, 0x1c00, 0x3250, 0, 0, wch_ch382_2s1p},
> > +       { 0x1c00, 0x3050, 0x1c00, 0x3050, 0, 0, wch_ch382_0s1p},  
> 
> Just a nit, wouldn't be better to preserve alphanumerical sorting?
> (Exchange lines in above pairs)
> 
> > +       [wch_ch382_0s1p] = {  
> ...
> >         [sunix_2s1p] = {  
> 
> Similar here, though it's not visible in the patch.
> 



-- 
Best regards,
 Alexander Gerasiov

 Contacts:
 e-mail: gq@...msu.su  Homepage: http://gerasiov.net  Skype: gerasiov
 PGP fingerprint: 04B5 9D90 DF7C C2AB CD49  BAEA CA87 E9E8 2AAC 33F1