Messages in this thread | | | Date | Tue, 31 May 2011 10:01:00 -0500 (CDT) | From | Manoj Iyer <> | Subject | Re: [PATCH] mmc: Enable MMC card reader for RICOH [1180:e823] |
| |
On Thu, 26 May 2011, Chris Ball wrote:
> Hi Manoj, > > On Thu, May 26 2011, Manoj Iyer wrote: >> Signed-off-by: Manoj Iyer <manoj.iyer@canonical.com> >> --- >> drivers/pci/quirks.c | 2 ++ >> include/linux/pci_ids.h | 1 + >> 2 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c >> index e8a1406..02145e9 100644 >> --- a/drivers/pci/quirks.c >> +++ b/drivers/pci/quirks.c >> @@ -2761,6 +2761,8 @@ static void ricoh_mmc_fixup_r5c832(struct pci_dev *dev) >> } >> DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_RICOH, PCI_DEVICE_ID_RICOH_R5C832, ricoh_mmc_fixup_r5c832); >> DECLARE_PCI_FIXUP_RESUME_EARLY(PCI_VENDOR_ID_RICOH, PCI_DEVICE_ID_RICOH_R5C832, ricoh_mmc_fixup_r5c832); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_RICOH, PCI_DEVICE_ID_RICOH_R5CE823, ricoh_mmc_fixup_r5c832); >> +DECLARE_PCI_FIXUP_RESUME_EARLY(PCI_VENDOR_ID_RICOH, PCI_DEVICE_ID_RICOH_R5CE823, ricoh_mmc_fixup_r5c832); >> #endif /*CONFIG_MMC_RICOH_MMC*/ >> >> #if defined(CONFIG_DMAR) || defined(CONFIG_INTR_REMAP) >> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h >> index 24787b7..2269dac 100644 >> --- a/include/linux/pci_ids.h >> +++ b/include/linux/pci_ids.h >> @@ -1537,6 +1537,7 @@ >> #define PCI_DEVICE_ID_RICOH_RL5C476 0x0476 >> #define PCI_DEVICE_ID_RICOH_RL5C478 0x0478 >> #define PCI_DEVICE_ID_RICOH_R5C822 0x0822 >> +#define PCI_DEVICE_ID_RICOH_R5CE823 0xe823 >> #define PCI_DEVICE_ID_RICOH_R5C832 0x0832 >> #define PCI_DEVICE_ID_RICOH_R5C843 0x0843 > > Do you not need a corresponding entry in drivers/mmc/host/sdhci-pci.c's > pci_device_id struct? >
I believe there is an entry there already, introduced by this patch: https://lkml.org/lkml/2011/2/11/309
> Thanks, > > - Chris. > -- > Chris Ball <cjb@laptop.org> <http://printf.net/> > One Laptop Per Child > >
-- ==================== Manoj Iyer Ubuntu/Canonical Hardware Enablement ====================
| |