[PATCH 1/8] i2c: imx: use struct representing i2c clk{div, val} pair

Lu Jingchang-B35083 B35083 at freescale.com
Fri Aug 9 04:12:25 EDT 2013


> -----Original Message-----
> From: Sascha Hauer [mailto:s.hauer at pengutronix.de]
> Sent: Thursday, August 08, 2013 1:54 AM
> To: Lu Jingchang-B35083
> Cc: wsa at the-dreams.de; shawn.guo at linaro.org; Estevam Fabio-R49496; linux-
> i2c at vger.kernel.org; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH 1/8] i2c: imx: use struct representing i2c clk{div,
> val} pair
> 
> Nice improvement of this series.
> 
> There are some unnecessary blank lines in the definitions in 7/8 and 8/8,
> but otherwise:
> 
> reviewed-by: Sascha Hauer <s.hauer at pengutronix.de>

[Lu Jingchang-B35083] 
Thank you very much for your review.
I leave one blank lines between definitions for readability, could you please help to comment 
the unnecessary blank lines on that patches so that I can remove them and send out the next 
version, thanks.





Best Regards,
Jingchang




More information about the linux-arm-kernel mailing list