All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] mmc: sdhci-pci: add another device id
@ 2013-06-20  9:57 Adrian Hunter
  2013-06-20  9:57 ` [PATCH 1/1] " Adrian Hunter
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Hunter @ 2013-06-20  9:57 UTC (permalink / raw)
  To: Chris Ball; +Cc: linux-mmc, Adrian Hunter

Hi

Here is a patch to add another PCI device id for
an eMMC host controller.  It is dependent on:

	mmc: sdhci-acpi: add support for eMMC hardware reset for HID 80860F14
	mmc: sdhci-pci: add support for eMMC hardware reset for BYT eMMC.

which were sent on 13 June.

Please queue all 3 for linux 3.11.


Adrian Hunter (1):
      mmc: sdhci-pci: add another device id

 drivers/mmc/host/sdhci-pci.c | 9 +++++++++
 1 file changed, 9 insertions(+)


Regards
Adrian

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH 1/1] mmc: sdhci-pci: add another device id
  2013-06-20  9:57 [PATCH 0/1] mmc: sdhci-pci: add another device id Adrian Hunter
@ 2013-06-20  9:57 ` Adrian Hunter
  2013-06-27 16:06   ` Chris Ball
  0 siblings, 1 reply; 4+ messages in thread
From: Adrian Hunter @ 2013-06-20  9:57 UTC (permalink / raw)
  To: Chris Ball; +Cc: linux-mmc, Adrian Hunter

Add another PCI device id for an eMMC host controller.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
 drivers/mmc/host/sdhci-pci.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c
index e082fac..d7d6bc8 100644
--- a/drivers/mmc/host/sdhci-pci.c
+++ b/drivers/mmc/host/sdhci-pci.c
@@ -36,6 +36,7 @@
 #define PCI_DEVICE_ID_INTEL_BYT_EMMC	0x0f14
 #define PCI_DEVICE_ID_INTEL_BYT_SDIO	0x0f15
 #define PCI_DEVICE_ID_INTEL_BYT_SD	0x0f16
+#define PCI_DEVICE_ID_INTEL_BYT_EMMC2	0x0f50
 
 /*
  * PCI registers
@@ -931,6 +932,14 @@ static const struct pci_device_id pci_ids[] = {
 	},
 
 	{
+		.vendor		= PCI_VENDOR_ID_INTEL,
+		.device		= PCI_DEVICE_ID_INTEL_BYT_EMMC2,
+		.subvendor	= PCI_ANY_ID,
+		.subdevice	= PCI_ANY_ID,
+		.driver_data	= (kernel_ulong_t)&sdhci_intel_byt_emmc,
+	},
+
+	{
 		.vendor		= PCI_VENDOR_ID_O2,
 		.device		= PCI_DEVICE_ID_O2_8120,
 		.subvendor	= PCI_ANY_ID,
-- 
1.7.11.7


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] mmc: sdhci-pci: add another device id
  2013-06-20  9:57 ` [PATCH 1/1] " Adrian Hunter
@ 2013-06-27 16:06   ` Chris Ball
  2013-06-28  7:20     ` Adrian Hunter
  0 siblings, 1 reply; 4+ messages in thread
From: Chris Ball @ 2013-06-27 16:06 UTC (permalink / raw)
  To: Adrian Hunter; +Cc: linux-mmc

Hi Adrian,

On Thu, Jun 20 2013, Adrian Hunter wrote:
> Add another PCI device id for an eMMC host controller.
>
> Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
> ---
>  drivers/mmc/host/sdhci-pci.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c
> index e082fac..d7d6bc8 100644
> --- a/drivers/mmc/host/sdhci-pci.c
> +++ b/drivers/mmc/host/sdhci-pci.c
> @@ -36,6 +36,7 @@
>  #define PCI_DEVICE_ID_INTEL_BYT_EMMC	0x0f14
>  #define PCI_DEVICE_ID_INTEL_BYT_SDIO	0x0f15
>  #define PCI_DEVICE_ID_INTEL_BYT_SD	0x0f16
> +#define PCI_DEVICE_ID_INTEL_BYT_EMMC2	0x0f50
>  
>  /*
>   * PCI registers
> @@ -931,6 +932,14 @@ static const struct pci_device_id pci_ids[] = {
>  	},
>  
>  	{
> +		.vendor		= PCI_VENDOR_ID_INTEL,
> +		.device		= PCI_DEVICE_ID_INTEL_BYT_EMMC2,
> +		.subvendor	= PCI_ANY_ID,
> +		.subdevice	= PCI_ANY_ID,
> +		.driver_data	= (kernel_ulong_t)&sdhci_intel_byt_emmc,
> +	},
> +
> +	{
>  		.vendor		= PCI_VENDOR_ID_O2,
>  		.device		= PCI_DEVICE_ID_O2_8120,
>  		.subvendor	= PCI_ANY_ID,

Thanks, pushed to mmc-next for 3.11.

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] mmc: sdhci-pci: add another device id
  2013-06-27 16:06   ` Chris Ball
@ 2013-06-28  7:20     ` Adrian Hunter
  0 siblings, 0 replies; 4+ messages in thread
From: Adrian Hunter @ 2013-06-28  7:20 UTC (permalink / raw)
  To: Chris Ball; +Cc: linux-mmc

On 27/06/13 19:06, Chris Ball wrote:
> Hi Adrian,
> 
> On Thu, Jun 20 2013, Adrian Hunter wrote:
>> Add another PCI device id for an eMMC host controller.
>>
>> Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
>> ---
>>  drivers/mmc/host/sdhci-pci.c | 9 +++++++++
>>  1 file changed, 9 insertions(+)
>>
>> diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c
>> index e082fac..d7d6bc8 100644
>> --- a/drivers/mmc/host/sdhci-pci.c
>> +++ b/drivers/mmc/host/sdhci-pci.c
>> @@ -36,6 +36,7 @@
>>  #define PCI_DEVICE_ID_INTEL_BYT_EMMC	0x0f14
>>  #define PCI_DEVICE_ID_INTEL_BYT_SDIO	0x0f15
>>  #define PCI_DEVICE_ID_INTEL_BYT_SD	0x0f16
>> +#define PCI_DEVICE_ID_INTEL_BYT_EMMC2	0x0f50
>>  
>>  /*
>>   * PCI registers
>> @@ -931,6 +932,14 @@ static const struct pci_device_id pci_ids[] = {
>>  	},
>>  
>>  	{
>> +		.vendor		= PCI_VENDOR_ID_INTEL,
>> +		.device		= PCI_DEVICE_ID_INTEL_BYT_EMMC2,
>> +		.subvendor	= PCI_ANY_ID,
>> +		.subdevice	= PCI_ANY_ID,
>> +		.driver_data	= (kernel_ulong_t)&sdhci_intel_byt_emmc,
>> +	},
>> +
>> +	{
>>  		.vendor		= PCI_VENDOR_ID_O2,
>>  		.device		= PCI_DEVICE_ID_O2_8120,
>>  		.subvendor	= PCI_ANY_ID,
> 
> Thanks, pushed to mmc-next for 3.11.

Thanks, but the patch on mmc-next has been mixed with someone else's patch
i.e. it now looks like this:

commit 058f4266c105df59b036625f9afbe96d3fc3ec1d
Author: Adrian Hunter <adrian.hunter@intel.com>
Date:   Thu Jun 20 12:57:59 2013 +0300

    mmc: sdhci-pci: add another device id

    Add another PCI device id for an eMMC host controller.

    Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
    Signed-off-by: Chris Ball <cjb@laptop.org>
---
 drivers/mmc/host/sdhci-of-esdhc.c | 17 +++++++++++++++++
 drivers/mmc/host/sdhci-pci.c      |  9 +++++++++
 include/linux/mmc/sdhci.h         |  2 ++
 3 files changed, 28 insertions(+)

diff --git a/drivers/mmc/host/sdhci-of-esdhc.c
b/drivers/mmc/host/sdhci-of-esdhc.c
index b2a635e..15039e2 100644
--- a/drivers/mmc/host/sdhci-of-esdhc.c
+++ b/drivers/mmc/host/sdhci-of-esdhc.c
@@ -121,6 +121,13 @@ static void esdhc_writeb(struct sdhci_host *host, u8
val, int reg)
 	if (reg == SDHCI_HOST_CONTROL) {
 		u32 dma_bits;

+		/*
+		 * If host control register is not standard, exit
+		 * this function
+		 */
+		if (host->quirks2 & SDHCI_QUIRK2_BROKEN_HOST_CONTROL)
+			return;
+
 		/* DMA select is 22,23 bits in Protocol Control Register */
 		dma_bits = (val & SDHCI_CTRL_DMA_MASK) << 5;
 		clrsetbits_be32(host->ioaddr + reg , SDHCI_CTRL_DMA_MASK << 5,
@@ -289,6 +296,7 @@ static const struct sdhci_pltfm_data sdhci_esdhc_pdata = {
 static int sdhci_esdhc_probe(struct platform_device *pdev)
 {
 	struct sdhci_host *host;
+	struct device_node *np;
 	int ret;

 	host = sdhci_pltfm_init(pdev, &sdhci_esdhc_pdata, 0);
@@ -297,6 +305,15 @@ static int sdhci_esdhc_probe(struct platform_device *pdev)

 	sdhci_get_of_property(pdev);

+	np = pdev->dev.of_node;
+	if (of_device_is_compatible(np, "fsl,p2020-esdhc")) {
+		/*
+		 * Freescale messed up with P2020 as it has a non-standard
+		 * host control register
+		 */
+		host->quirks2 |= SDHCI_QUIRK2_BROKEN_HOST_CONTROL;
+	}
+
 	/* call to generic mmc_of_parse to support additional capabilities */
 	mmc_of_parse(host->mmc);

diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c
index e082fac..d7d6bc8 100644
--- a/drivers/mmc/host/sdhci-pci.c
+++ b/drivers/mmc/host/sdhci-pci.c
@@ -36,6 +36,7 @@
 #define PCI_DEVICE_ID_INTEL_BYT_EMMC	0x0f14
 #define PCI_DEVICE_ID_INTEL_BYT_SDIO	0x0f15
 #define PCI_DEVICE_ID_INTEL_BYT_SD	0x0f16
+#define PCI_DEVICE_ID_INTEL_BYT_EMMC2	0x0f50

 /*
  * PCI registers
@@ -931,6 +932,14 @@ static const struct pci_device_id pci_ids[] = {
 	},

 	{
+		.vendor		= PCI_VENDOR_ID_INTEL,
+		.device		= PCI_DEVICE_ID_INTEL_BYT_EMMC2,
+		.subvendor	= PCI_ANY_ID,
+		.subdevice	= PCI_ANY_ID,
+		.driver_data	= (kernel_ulong_t)&sdhci_intel_byt_emmc,
+	},
+
+	{
 		.vendor		= PCI_VENDOR_ID_O2,
 		.device		= PCI_DEVICE_ID_O2_8120,
 		.subvendor	= PCI_ANY_ID,
diff --git a/include/linux/mmc/sdhci.h b/include/linux/mmc/sdhci.h
index a745180..e3c6a74 100644
--- a/include/linux/mmc/sdhci.h
+++ b/include/linux/mmc/sdhci.h
@@ -96,6 +96,8 @@ struct sdhci_host {
 #define SDHCI_QUIRK2_NO_1_8_V				(1<<2)
 #define SDHCI_QUIRK2_PRESET_VALUE_BROKEN		(1<<3)
 #define SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON		(1<<4)
+/* Controller has a non-standard host control register */
+#define SDHCI_QUIRK2_BROKEN_HOST_CONTROL		(1<<5)

 	int irq;		/* Device IRQ */
 	void __iomem *ioaddr;	/* Mapped address */


^ permalink raw reply related	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-06-28  7:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-20  9:57 [PATCH 0/1] mmc: sdhci-pci: add another device id Adrian Hunter
2013-06-20  9:57 ` [PATCH 1/1] " Adrian Hunter
2013-06-27 16:06   ` Chris Ball
2013-06-28  7:20     ` Adrian Hunter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.