> -----Original Message----- > From: Sascha Hauer [mailto:s.hauer@xxxxxxxxxxxxxx] > Sent: Thursday, August 08, 2013 1:54 AM > To: Lu Jingchang-B35083 > Cc: wsa@xxxxxxxxxxxxx; shawn.guo@xxxxxxxxxx; Estevam Fabio-R49496; linux- > i2c@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > 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@xxxxxxxxxxxxxx> [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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
