Re: [PATCH 1/5] PCI: add ArrowLake-S PCI ID for Intel HDAudio subsystem.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 02 Aug 2023 17:52:26 +0200,
Bjorn Helgaas wrote:
> 
> On Wed, Aug 02, 2023 at 10:01:01AM -0500, Pierre-Louis Bossart wrote:
> > Add part ID to common include file
> 
> Please drop period at end of subject and add one at the end of the
> commit log.
> 
> Also mention the drivers that will use this new #define; looks like
> hda_intel.c and ...
> 
> Well, actually, I only see that one use, which means we probably
> shouldn't add this #define to pci_ids.h, per the comment at the top of
> the file.  If there's only one use, use the hex ID in the driver (or
> add a #define in the driver itself).

Judging from the previous patterns, the same ID could be required for
ASoC SOF driver, too, which isn't included in this patch set.  In
that case, it's worth to put to pci_ids.h.
(OTOH, it can be done at a later stage, too.)


thanks,

Takashi



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux