On Thu, Dec 19, 2013 at 07:08:16AM +0000, Anson.Huang@xxxxxxxxxxxxx wrote: > >On Thu, Dec 19, 2013 at 01:17:24PM -0500, Anson Huang wrote: > >> @@ -427,6 +429,13 @@ static int imx_thermal_probe(struct platform_device > >*pdev) > >> return ret; > >> } > >> > >> + data->thermal_clk = devm_clk_get(&pdev->dev, NULL); > >> + if (IS_ERR(data->thermal_clk)) { > >> + ret = IS_ERR(data->thermal_clk); > >> + dev_err(&pdev->dev, "failed to get thermal clk!\n"); > >> + return ret; > >> + } > >> + > > > >So when the new kernel runs on a board with an old DTB installed, thermal > >driver will be broken. > > > Yes, I thought about this case, but the previous implement is incorrect, if > the PLL3 is not enabled by other drivers, thermal driver will not work, so > this patch is a bug fix, not enhancement. So we still need to consider old > dts case? The thing is mainline kernel runs on many board with thermal driver being functional today. That said, PLL3 is already enabled on these platforms when thermal driver is running. You cannot fix a bug but meanwhile break these users who use old DTB. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html