Quoting Thierry Reding (2013-02-13 07:53:13) > On Tue, Feb 12, 2013 at 12:34:02PM -0700, Stephen Warren wrote: > > From: Stephen Warren <swarren@xxxxxxxxxx> > > > > 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@xxxxxxxxxx> > > --- > > 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@xxxxxxxxxxxxxxxxx> > Tested-by: Thierry Reding <thierry.reding@xxxxxxxxxxxxxxxxx> Hurray for testing. If this hasn't already been sent to Arnd & Olof: Acked-by: Mike Turquette <mturquette@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html