Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755818AbbFOJzi (ORCPT ); Mon, 15 Jun 2015 05:55:38 -0400 Received: from mail-qg0-f44.google.com ([209.85.192.44]:32953 "EHLO mail-qg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755736AbbFOJzL (ORCPT ); Mon, 15 Jun 2015 05:55:11 -0400 MIME-Version: 1.0 In-Reply-To: <1434024707-6245-3-git-send-email-Vincent.Wan@amd.com> References: <1434024707-6245-1-git-send-email-Vincent.Wan@amd.com> <1434024707-6245-3-git-send-email-Vincent.Wan@amd.com> Date: Mon, 15 Jun 2015 11:55:08 +0200 Message-ID: Subject: Re: [PATCH V2 3/3] SDHCI: Change AMD SDHCI quirk application scope From: Ulf Hansson To: Wan ZongShun Cc: linux-mmc , "linux-i2c@vger.kernel.org" , bp@alien8.de, Jean Delvare , Linux PCI , "linux-kernel@vger.kernel.org" , ZongShun Wan Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2401 Lines: 69 On 11 June 2015 at 14:11, Wan ZongShun wrote: > Change this quirk to apply to AMD Carrizo platform. > > Signed-off-by: Wan ZongShun > > Tested-by: Nath, Arindam > Tested-by: Ramesh, Ramya Thanks, applied! I changed the prefix of the commit message header from "SDHCI." to "mmc: sdhci-pci:" Kind regards Uffe > --- > drivers/mmc/host/sdhci-pci.c | 25 ++++++++++++++++++++++++- > 1 file changed, 24 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c > index f208f20..94f54d2 100644 > --- a/drivers/mmc/host/sdhci-pci.c > +++ b/drivers/mmc/host/sdhci-pci.c > @@ -724,14 +724,37 @@ static const struct sdhci_pci_fixes sdhci_rtsx = { > .probe_slot = rtsx_probe_slot, > }; > > +/*AMD chipset generation*/ > +enum amd_chipset_gen { > + AMD_CHIPSET_BEFORE_ML, > + AMD_CHIPSET_CZ, > + AMD_CHIPSET_NL, > + AMD_CHIPSET_UNKNOWN, > +}; > + > static int amd_probe(struct sdhci_pci_chip *chip) > { > struct pci_dev *smbus_dev; > + enum amd_chipset_gen gen; > > smbus_dev = pci_get_device(PCI_VENDOR_ID_AMD, > PCI_DEVICE_ID_AMD_HUDSON2_SMBUS, NULL); > + if (smbus_dev) { > + gen = AMD_CHIPSET_BEFORE_ML; > + } else { > + smbus_dev = pci_get_device(PCI_VENDOR_ID_AMD, > + PCI_DEVICE_ID_AMD_KERNCZ_SMBUS, NULL); > + if (smbus_dev) { > + if (smbus_dev->revision < 0x51) > + gen = AMD_CHIPSET_CZ; > + else > + gen = AMD_CHIPSET_NL; > + } else { > + gen = AMD_CHIPSET_UNKNOWN; > + } > + } > > - if (smbus_dev && (smbus_dev->revision < 0x51)) { > + if ((gen == AMD_CHIPSET_BEFORE_ML) || (gen == AMD_CHIPSET_CZ)) { > chip->quirks2 |= SDHCI_QUIRK2_CLEAR_TRANSFERMODE_REG_BEFORE_CMD; > chip->quirks2 |= SDHCI_QUIRK2_BROKEN_HS200; > } > -- > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/