From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [RFC/PATCH 3/3] ACPI based I/O APIC hot-plug Date: Fri, 22 Apr 2005 08:15:17 -0700 Message-ID: <20050422151517.GA29039@kroah.com> References: <4267AD21.7040006@jp.fujitsu.com> <1114104131.2784.43.camel@eeyore> <42689BEB.90401@jp.fujitsu.com> <1114181881.4902.4.camel@eeyore> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1114181881.4902.4.camel@eeyore> Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Bjorn Helgaas Cc: Kenji Kaneshige , Andrew Morton , Len Brown , "Luck, Tony" , acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, linux-ia64-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, pcihpd-discuss-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-acpi@vger.kernel.org On Fri, Apr 22, 2005 at 08:58:01AM -0600, Bjorn Helgaas wrote: > On Fri, 2005-04-22 at 15:38 +0900, Kenji Kaneshige wrote: > > Bjorn Helgaas wrote: > > > > > >>+ if ((dev->class >> 8) != PCI_CLASS_SYSTEM_PIC) > > >>+ continue; > > >>+ if ((dev->class & 0xff) == 0x10 || (dev->class & 0xff) == 0x20) > > > > > > > > > What are 0x10 and 0x20? Looks like they should be #defines in > > > include/linux/pci_ids.h. > > > > 0x10 and 0x20 are programing interfaces for I/O APIC and I/O xAPIC > > respectively. #define for these values looks good. But I don't know > > if I can put new #defines into pci_ids.h and how to name them because > > I could not find the header file (including pci_ids.h) that #defines > > the values for programming interfaces. So I want to add the comments > > to explain these values (0x10, 0x20) instead of adding new #defines into > > pci_ids.h for now. > > I think your patch should just add the values to pci_ids.h. If > somebody doesn't like that, he or she will complain and you can fall > back to just using 0x10 and 0x20. But I suspect it will be fine. Yes, I have no problem with that. thanks, greg k-h ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click