> -----Original Message----- > From: Sascha Hauer [mailto:[email protected]] > Sent: Thursday, August 08, 2013 1:54 AM > To: Lu Jingchang-B35083 > Cc: [email protected]; [email protected]; Estevam Fabio-R49496; linux- > [email protected]; [email protected] > 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 <[email protected]>
[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 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
