[U-Boot] [RESEND PATCH v3 1/2] mmc: rockchip_sdhci: add clock init for mmc
Kever Yang
kever.yang at rock-chips.com
Thu Dec 29 01:53:51 CET 2016
Hi Stefan,
Thanks for your review comment.
On 12/29/2016 02:35 AM, Stefan Herbrechtsmeier wrote:
> Hi,
>
> Am 28.12.2016 um 12:01 schrieb Jaehoon Chung:
>> On 12/28/2016 12:32 PM, Kever Yang wrote:
>>> Init the clock rate to max-frequency from dts with clock driver api.
>>>
>>> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
>> Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>
> This is an incorrect use of the max-frequency property.
>
> The max-frequency value limit the output clock of the mmc interface
> and depends on the controller, circuit (level shifter), board and so
> on. It doesn't represents the clock frequency of the controller.
>
> The clock setup inside the clock framework should use the
> assigned-clock-rates property. The mmc driver should only enable the
> clock and pass the clock rate together with the max-frequency to the
> mmc framework.
I'm not good at mmc controller and driver framework, but seems that the
sdhci core treats the max-frequency as the clock input from clock
module, right?
What if the mmc controller max-frequency is not equal to the clock
module output which is possible? Does kernel deal with this, and how.
Thanks,
- Kever
>
> Regards,
> Stefan
>
>
>
>
More information about the U-Boot
mailing list