Hi, a comment: On Wed, 31 Aug 2011, Keerthy wrote: > div_ts_ck feeds only the temperature sensor functional clock > and also has a clksel associated (for divider selection). Mapping this > as the functional clock for the temperature sensor in clkdev table, > so a clk_set_rate() in the driver would have the effect of changing the > temperature sensor clock rate indirectly. > > Signed-off-by: Keerthy <j-keerthy@xxxxxx> > Reviewed-by: Rajendra Nayak <rnayak@xxxxxx> > Cc: tony@xxxxxxxxxxx > Cc: rnayak@xxxxxx > --- > arch/arm/mach-omap2/clock44xx_data.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c > index 2af0e3f..4a788f4 100644 > --- a/arch/arm/mach-omap2/clock44xx_data.c > +++ b/arch/arm/mach-omap2/clock44xx_data.c > @@ -3187,7 +3187,7 @@ static struct omap_clk omap44xx_clks[] = { > CLK(NULL, "bandgap_fclk", &bandgap_fclk, CK_443X), > CLK(NULL, "bandgap_ts_fclk", &bandgap_ts_fclk, CK_446X), > CLK(NULL, "des3des_fck", &des3des_fck, CK_443X), > - CLK(NULL, "div_ts_ck", &div_ts_ck, CK_446X), > + CLK("omap_temp_sensor.0", "fck", &div_ts_ck, CK_446X), It shouldn't be necessary to add the device name/ID here. We're trying to get rid of these. The omap_device code should take care of adding the appropriate alias. > CLK(NULL, "dmic_sync_mux_ck", &dmic_sync_mux_ck, CK_443X), > CLK(NULL, "dmic_fck", &dmic_fck, CK_443X), > CLK(NULL, "dsp_fck", &dsp_fck, CK_443X), > -- > 1.7.0.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html