On 06-07-16, 12:20, weiyj_lk@xxxxxxx wrote: > From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> > > In case of error, the function clk_get_parent() returns NULL pointer > not ERR_PTR(). The IS_ERR() test in the return value check should be > replaced with NULL test. NULL is a valid clock as per the clk-API and so this patch is wrong. You need to investigate why you are getting NULL here for your platform. -- viresh