RE: [Intel-wired-lan] [PATCH iwl-next v2 1/2] PCI: Add PCI_VDEVICE_SUB helper macro
From: Romanowski, Rafal
Date: Wed Oct 30 2024 - 04:19:20 EST
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of Simon
> Horman
> Sent: Tuesday, October 22, 2024 6:04 PM
> To: Bjorn Helgaas <helgaas@xxxxxxxxxx>
> Cc: Kwapulinski, Piotr <piotr.kwapulinski@xxxxxxxxx>; intel-wired-
> lan@xxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; bhelgaas@xxxxxxxxxx; linux-
> pci@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Kitszel, Przemyslaw
> <przemyslaw.kitszel@xxxxxxxxx>
> Subject: Re: [Intel-wired-lan] [PATCH iwl-next v2 1/2] PCI: Add
> PCI_VDEVICE_SUB helper macro
>
> On Tue, Oct 22, 2024 at 10:30:11AM -0500, Bjorn Helgaas wrote:
> > On Mon, Oct 21, 2024 at 04:46:54PM +0200, Piotr Kwapulinski wrote:
> > > PCI_VDEVICE_SUB generates the pci_device_id struct layout for the
> > > specific PCI device/subdevice. Private data may follow the output.
> > >
> > > Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@xxxxxxxxx>
> > > Signed-off-by: Piotr Kwapulinski <piotr.kwapulinski@xxxxxxxxx>
> >
> > This looks OK to me but needs to be included in a series that uses it.
> > I looked this message up on lore but can't find the 2/2 patch that
> > presumably uses it.
> >
> > If 2/2 uses this,
> >
> > Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>
> Hi Bjorn,
>
> The threading of this patch-set does seem somehow broken.
> But, FWIIW, I believe that patch 2/2 is here:
>
> - [PATCH iwl-next v2 2/2] ixgbevf: Add support for Intel(R) E610 device
> https://lore.kernel.org/netdev/20241021144841.5476-1-
> piotr.kwapulinski@xxxxxxxxx/
Tested-by: Rafal Romanowski <rafal.romanowski@xxxxxxxxx>