Re: [PATCH] clk: tegra: fix driver to match DT binding

[Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1024 bytes --]

On Tue, Feb 12, 2013 at 12:34:02PM -0700, Stephen Warren wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> enum tegra*_clk is intended to match the IDs listed in the Tegra clock
> bindings. There are a few mismatches, which this patch fixes:
> 
> 1) pll_s and cop were left out of the Tegra20 enum.
> 
> 2) spdif_in and spdif_out were swapped relative to the Tegra30 binding.
> 
> 3) i2cslow was misnamed as i2c_slow, and a duplicate i2cslow clock added
>    to the Tegra30 enum.
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  drivers/clk/tegra/clk-tegra20.c |    4 ++--
>  drivers/clk/tegra/clk-tegra30.c |    6 +++---
>  2 files changed, 5 insertions(+), 5 deletions(-)

Looks good to me and I haven't seen any regressions, so:

Reviewed-by: Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>
Tested-by: Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]