On Friday, November 23, 2012 12:23:11 PM Adrian Hunter wrote: > On 22/11/12 23:24, Rafael J. Wysocki wrote: > > On Thursday, November 22, 2012 04:46:10 PM Adrian Hunter wrote: > >> On 22/11/12 15:55, Chris Ball wrote: > >>> Hi, > >>> > >>> On Thu, Nov 22 2012, Adrian Hunter wrote: > >>>> Here is SDHCI ACPI driver. It is dependent on new ACPI Platform support > >>>> so I suggest Rafael takes the patches with Chris' Ack. > >>>> > >>>> Please note that I would prefer this to be queued for 3.8 > >>> > >>> Looks fine: > >>> > >>> Acked-by: Chris Ball <cjb@xxxxxxxxxx> > >> > >> Thank you! > >> > >>> > >>> I have some dumb questions, though -- what kind of platforms ship with > >>> these devices? Do they ever have the controller on PCI too, and what > >>> happens with sdhci-pci vs. sdhci-acpi in that case? > >> > >> Since the arrival of ACPI5, platform devices can be configured using ACPI > >> tables. PCI can also be used, but the firmware ensures that the same > >> device is not enumerated via both ACPI and PCI. > >> > >> Rafael can you take these patches? > > > > Well, I'd prefer pnpacpi/core.c to actually use acpi_platform_device_ids[] > > directly in addition to excluded_id_list[], so that duplicate entries don't > > have to be added to the both of them. > > > > Also, I wonder if you really don't want to use ACPI PM and if you don't, > > then why? > > Mika and Lv Zheng are working on adding it to acpi_platform OK Please address the Mika's comment for [3/3], though. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html