Best Regards. Anson huang 黄勇才 Freescale Semiconductor Shanghai 上海浦东新区亮景路192号A座2楼 201203 Tel:021-28937058
>-----Original Message----- >From: Shawn Guo [mailto:shawn....@linaro.org] >Sent: Thursday, December 19, 2013 2:59 PM >To: Huang Yongcai-B20788 >Cc: ker...@pengutronix.de; rui.zh...@intel.com; eduardo.valen...@ti.com; >devicetree@vger.kernel.org; linux-...@vger.kernel.org; linux-arm- >ker...@lists.infradead.org; linux...@vger.kernel.org >Subject: Re: [PATCH 2/3] thermal: imx: add necessary clk operation > >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? Anson >Shawn N�����r��y����b�X��ǧv�^�){.n�+���z��z��z)����w*jg��������ݢj/���z�ޖ��2�ޙ����&�)ߡ�a�����G���h��j:+v���w��٥