Re: [PATCH] clk: tegra: Implement memory-controller clock

From: Mikko Perttunen
Date: Mon Aug 04 2014 - 02:35:31 EST


I'm curious why you used emc_mux as the parent clock? (I don't really understand why this clock exists, and my EMC series removes it). Using 'emc' would be compatible with the EMC series :)

A minor one: the MC clock should probably be always enabled.

Apart from these:

Reviewed-by: Mikko Perttunen <mperttunen@xxxxxxxxxx>
Tested-by: Mikko Perttunen <mperttunen@xxxxxxxxxx>

Cheers,
Mikko

On 01/08/14 15:09, Thierry Reding wrote:
From: Thierry Reding <treding@xxxxxxxxxx>

The memory controller clock runs either at half or the same frequency as
the EMC clock.

Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/