Re: [PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-12-11 Thread Shawn Guo
On Tue, Dec 11, 2012 at 08:33:34AM -0200, Fabio Estevam wrote: > Shawn/Mike, > > On Wed, Nov 21, 2012 at 7:33 PM, Fabio Estevam wrote: > > From: Fabio Estevam > > > > mxs platform has been converted to device tree. > > > > There is no need to check if np is NULL after doing: > > > > np =

Re: [PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-12-11 Thread Fabio Estevam
Shawn/Mike, On Wed, Nov 21, 2012 at 7:33 PM, Fabio Estevam wrote: > From: Fabio Estevam > > mxs platform has been converted to device tree. > > There is no need to check if np is NULL after doing: > > np = of_find_compatible_node(NULL, NULL, "fsl,imx[23/28]-clkctrl"); > > ,as it will always be

Re: [PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-12-11 Thread Fabio Estevam
Shawn/Mike, On Wed, Nov 21, 2012 at 7:33 PM, Fabio Estevam feste...@gmail.com wrote: From: Fabio Estevam fabio.este...@freescale.com mxs platform has been converted to device tree. There is no need to check if np is NULL after doing: np = of_find_compatible_node(NULL, NULL,

Re: [PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-12-11 Thread Shawn Guo
On Tue, Dec 11, 2012 at 08:33:34AM -0200, Fabio Estevam wrote: Shawn/Mike, On Wed, Nov 21, 2012 at 7:33 PM, Fabio Estevam feste...@gmail.com wrote: From: Fabio Estevam fabio.este...@freescale.com mxs platform has been converted to device tree. There is no need to check if np is NULL

[PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-11-22 Thread Fabio Estevam
From: Fabio Estevam mxs platform has been converted to device tree. There is no need to check if np is NULL after doing: np = of_find_compatible_node(NULL, NULL, "fsl,imx[23/28]-clkctrl"); ,as it will always be non-NULL. Signed-off-by: Fabio Estevam --- drivers/clk/mxs/clk-imx23.c |8

[PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-11-22 Thread Fabio Estevam
From: Fabio Estevam fabio.este...@freescale.com mxs platform has been converted to device tree. There is no need to check if np is NULL after doing: np = of_find_compatible_node(NULL, NULL, fsl,imx[23/28]-clkctrl); ,as it will always be non-NULL. Signed-off-by: Fabio Estevam