Re: [PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-30 Thread Ulf Hansson 
On 29 March 2017 at 20:16, Azhar Shaikh  wrote:
> Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
> controllers.
>
> Signed-off-by: Azhar Shaikh 


Thanks, applied for next!

Kind regards
Uffe


> ---
> Changes in v2:
>   - Rebased the patch on top of 'next' branch.
>   - No code change.
>
>  drivers/mmc/host/sdhci-acpi.c | 6 ++
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
> index 665c5f3..c6a9a1b 100644
> --- a/drivers/mmc/host/sdhci-acpi.c
> +++ b/drivers/mmc/host/sdhci-acpi.c
> @@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>
> /* Platform specific code during sd probe slot goes here */
>
> -   if (hid && !strcmp(hid, "80865ACA")) {
> +   if (hid && !strcmp(hid, "80865ACA"))
> host->mmc_host_ops.get_cd = bxt_get_cd;
> -   host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
> -   }
>
> return 0;
>  }
> @@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
> .quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
> .quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
>SDHCI_QUIRK2_STOP_WITH_TC,
> -   .caps= MMC_CAP_WAIT_WHILE_BUSY,
> +   .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
> .probe_slot = sdhci_acpi_sd_probe_slot,
>  };
>
> --
> 1.9.1
>

Re: [PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-30 Thread Ulf Hansson 
On 29 March 2017 at 20:16, Azhar Shaikh  wrote:
> Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
> controllers.
>
> Signed-off-by: Azhar Shaikh 


Thanks, applied for next!

Kind regards
Uffe


> ---
> Changes in v2:
>   - Rebased the patch on top of 'next' branch.
>   - No code change.
>
>  drivers/mmc/host/sdhci-acpi.c | 6 ++
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
> index 665c5f3..c6a9a1b 100644
> --- a/drivers/mmc/host/sdhci-acpi.c
> +++ b/drivers/mmc/host/sdhci-acpi.c
> @@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>
> /* Platform specific code during sd probe slot goes here */
>
> -   if (hid && !strcmp(hid, "80865ACA")) {
> +   if (hid && !strcmp(hid, "80865ACA"))
> host->mmc_host_ops.get_cd = bxt_get_cd;
> -   host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
> -   }
>
> return 0;
>  }
> @@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
> .quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
> .quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
>SDHCI_QUIRK2_STOP_WITH_TC,
> -   .caps= MMC_CAP_WAIT_WHILE_BUSY,
> +   .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
> .probe_slot = sdhci_acpi_sd_probe_slot,
>  };
>
> --
> 1.9.1
>

Re: [PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-30 Thread Adrian Hunter 
On 29/03/17 21:16, Azhar Shaikh wrote:
> Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
> controllers.
> 
> Signed-off-by: Azhar Shaikh 

Acked-by: Adrian Hunter 

> ---
> Changes in v2:
>   - Rebased the patch on top of 'next' branch.
>   - No code change.
> 
>  drivers/mmc/host/sdhci-acpi.c | 6 ++
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
> index 665c5f3..c6a9a1b 100644
> --- a/drivers/mmc/host/sdhci-acpi.c
> +++ b/drivers/mmc/host/sdhci-acpi.c
> @@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>  
>   /* Platform specific code during sd probe slot goes here */
>  
> - if (hid && !strcmp(hid, "80865ACA")) {
> + if (hid && !strcmp(hid, "80865ACA"))
>   host->mmc_host_ops.get_cd = bxt_get_cd;
> - host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
> - }
>  
>   return 0;
>  }
> @@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>   .quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
>   .quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
>  SDHCI_QUIRK2_STOP_WITH_TC,
> - .caps= MMC_CAP_WAIT_WHILE_BUSY,
> + .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
>   .probe_slot = sdhci_acpi_sd_probe_slot,
>  };
>  
>

Re: [PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-30 Thread Adrian Hunter 
On 29/03/17 21:16, Azhar Shaikh wrote:
> Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
> controllers.
> 
> Signed-off-by: Azhar Shaikh 

Acked-by: Adrian Hunter 

> ---
> Changes in v2:
>   - Rebased the patch on top of 'next' branch.
>   - No code change.
> 
>  drivers/mmc/host/sdhci-acpi.c | 6 ++
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
> index 665c5f3..c6a9a1b 100644
> --- a/drivers/mmc/host/sdhci-acpi.c
> +++ b/drivers/mmc/host/sdhci-acpi.c
> @@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>  
>   /* Platform specific code during sd probe slot goes here */
>  
> - if (hid && !strcmp(hid, "80865ACA")) {
> + if (hid && !strcmp(hid, "80865ACA"))
>   host->mmc_host_ops.get_cd = bxt_get_cd;
> - host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
> - }
>  
>   return 0;
>  }
> @@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct 
> platform_device *pdev,
>   .quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
>   .quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
>  SDHCI_QUIRK2_STOP_WITH_TC,
> - .caps= MMC_CAP_WAIT_WHILE_BUSY,
> + .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
>   .probe_slot = sdhci_acpi_sd_probe_slot,
>  };
>  
>

[PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-29 Thread Azhar Shaikh 
Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
controllers.

Signed-off-by: Azhar Shaikh 
---
Changes in v2:
  - Rebased the patch on top of 'next' branch.
  - No code change.

 drivers/mmc/host/sdhci-acpi.c | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
index 665c5f3..c6a9a1b 100644
--- a/drivers/mmc/host/sdhci-acpi.c
+++ b/drivers/mmc/host/sdhci-acpi.c
@@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct platform_device 
*pdev,
 
/* Platform specific code during sd probe slot goes here */
 
-   if (hid && !strcmp(hid, "80865ACA")) {
+   if (hid && !strcmp(hid, "80865ACA"))
host->mmc_host_ops.get_cd = bxt_get_cd;
-   host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
-   }
 
return 0;
 }
@@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct platform_device 
*pdev,
.quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
.quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
   SDHCI_QUIRK2_STOP_WITH_TC,
-   .caps= MMC_CAP_WAIT_WHILE_BUSY,
+   .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
.probe_slot = sdhci_acpi_sd_probe_slot,
 };
 
-- 
1.9.1

[PATCH v2 1/2] mmc: sdhci-acpi: Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel controllers

2017-03-29 Thread Azhar Shaikh 
Set MMC_CAP_AGGRESSIVE_PM for BYT-related Intel SD card
controllers.

Signed-off-by: Azhar Shaikh 
---
Changes in v2:
  - Rebased the patch on top of 'next' branch.
  - No code change.

 drivers/mmc/host/sdhci-acpi.c | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
index 665c5f3..c6a9a1b 100644
--- a/drivers/mmc/host/sdhci-acpi.c
+++ b/drivers/mmc/host/sdhci-acpi.c
@@ -263,10 +263,8 @@ static int sdhci_acpi_sd_probe_slot(struct platform_device 
*pdev,
 
/* Platform specific code during sd probe slot goes here */
 
-   if (hid && !strcmp(hid, "80865ACA")) {
+   if (hid && !strcmp(hid, "80865ACA"))
host->mmc_host_ops.get_cd = bxt_get_cd;
-   host->mmc->caps |= MMC_CAP_AGGRESSIVE_PM;
-   }
 
return 0;
 }
@@ -302,7 +300,7 @@ static int sdhci_acpi_sd_probe_slot(struct platform_device 
*pdev,
.quirks  = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
.quirks2 = SDHCI_QUIRK2_CARD_ON_NEEDS_BUS_ON |
   SDHCI_QUIRK2_STOP_WITH_TC,
-   .caps= MMC_CAP_WAIT_WHILE_BUSY,
+   .caps= MMC_CAP_WAIT_WHILE_BUSY | MMC_CAP_AGGRESSIVE_PM,
.probe_slot = sdhci_acpi_sd_probe_slot,
 };
 
-- 
1.9.1