linux-kernel - Re: [PATCH v1 03/13] serial: 8250_exar: Kill CTI_PCI

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240502113610.01881f3b@SWDEV2.connecttech.local>
Date: Thu, 2 May 2024 11:36:10 -0400
From: Parker Newman <parker@...est.io>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Parker Newman <pnewman@...necttech.com>, linux-kernel@...r.kernel.org,
 linux-serial@...r.kernel.org, Greg Kroah-Hartman
 <gregkh@...uxfoundation.org>, Jiri Slaby <jirislaby@...nel.org>
Subject: Re: [PATCH v1 03/13] serial: 8250_exar: Kill CTI_PCI_DEVICE()

On Thu, 2 May 2024 18:29:05 +0300
Andy Shevchenko <andriy.shevchenko@...ux.intel.com> wrote:

> On Thu, May 02, 2024 at 11:13:14AM -0400, Parker Newman wrote:
> > On Thu,  2 May 2024 17:43:57 +0300
> > Andy Shevchenko <andriy.shevchenko@...ux.intel.com> wrote:
>
> > > The CTI_PCI_DEVICE() duplicates EXAR_DEVICE(). Kill the former.
>
> ...
>
> > > -// For Connect Tech cards with Connect Tech vendor/device PCI IDs (FPGA based)
> > > -#define CTI_PCI_DEVICE(devid, bd) {                     \
> > > -	PCI_DEVICE_SUB(                                 \
> > > -		PCI_VENDOR_ID_CONNECT_TECH,             \
> > > -		PCI_DEVICE_ID_CONNECT_TECH_PCI_##devid, \
>
>
> #define PCI_DEVICE_SUB(vend, dev, subvend, subdev) \
>         .vendor = (vend), .device = (dev), \
>         .subvendor = (subvend), .subdevice = (subdev)
>
> #define PCI_DEVICE_DATA(vend, dev, data) \
>         .vendor = PCI_VENDOR_ID_##vend, .device = PCI_DEVICE_ID_##vend##_##dev, \
>         .subvendor = PCI_ANY_ID, .subdevice = PCI_ANY_ID, 0, 0, \
>         .driver_data = (kernel_ulong_t)(data)
>
>
> > > -		PCI_ANY_ID,                             \
> > > -		PCI_ANY_ID), 0, 0,                      \
> > > -		(kernel_ulong_t)&bd                     \
> > > -	}
> > > -
> > >  #define EXAR_DEVICE(vend, devid, bd) { PCI_DEVICE_DATA(vend, devid, &bd) }
>
> > This is not correct. The CTI_PCI_DEVICE() macro is for cards that have the
> > Connect Tech PCI Vendor ID (not Sub-Vendor ID). EXAR_DEVICE() is for cards with
> > Exar PCI Vendor ID.
>
> Above I added current code of these macros, can you elaborate how it's incorrect?
>

Sorry, you are correct. I was mixed up.
Thanks,
Parker