Hi,

On 01/04/2018 06:40 PM, Siva Durga Prasad Paladugu wrote:
> This patch adds support to disable clock if clk_disable
> was set and then enable or set clock if the clock was changed
> or clock was disabled when clock needs to be enabled.
> 
> Signed-off-by: Siva Durga Prasad Paladugu <siva...@xilinx.com>
> ---
>  drivers/mmc/sdhci.c | 7 ++++++-
>  include/sdhci.h     | 1 +
>  2 files changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
> index e2ddf5d..2bd721f 100644
> --- a/drivers/mmc/sdhci.c
> +++ b/drivers/mmc/sdhci.c
> @@ -437,8 +437,13 @@ static int sdhci_set_ios(struct mmc *mmc)
>       if (host->ops && host->ops->set_control_reg)
>               host->ops->set_control_reg(host);
>  
> -     if (mmc->clock != host->clock)
> +     if ((mmc->clock != host->clock || host->clk_disabled) &&
> +         !mmc->clk_disable)
>               sdhci_set_clock(mmc, mmc->clock);
> +     if (mmc->clk_disable) {
> +             sdhci_set_clock(mmc, 0);
> +             host->clk_disabled = true;
> +     }

host->clk_disabled is really needs? Where is clk_disabled set to false?

Best Regards,
Jaehoon Chung

>  
>       /* Set bus width */
>       ctrl = sdhci_readb(host, SDHCI_HOST_CONTROL);
> diff --git a/include/sdhci.h b/include/sdhci.h
> index 7e84012..4fc4140 100644
> --- a/include/sdhci.h
> +++ b/include/sdhci.h
> @@ -259,6 +259,7 @@ struct sdhci_host {
>       uint    voltages;
>  
>       struct mmc_config cfg;
> +     bool clk_disabled;
>  };
>  
>  #ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
> 

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to