Oh ok, I'll revert that then, no problem. So, shall I consider everything else as accepted in the patch? Thank you very much for your time and your comments. I appreciate it. Regards, Sergio -----Original Message----- From: Paul Walmsley [mailto:paul@xxxxxxxxx] Sent: Wednesday, September 03, 2008 10:23 AM To: Aguirre Rodriguez, Sergio Alberto Cc: linux-omap@xxxxxxxxxxxxxxx; Aguirre Rodriguez, Sergio Alberto Subject: Re: [RESEND 2][Review Request] Adding csi2_fck declaration to clock34xx.h Hello Sergio, On Tue, 2 Sep 2008, Aguirre Rodriguez, Sergio Alberto wrote: > From: Sergio Aguirre <saaguirre@xxxxxx> > > ARM: OMAP: Add CSI2 clock struct for handling it with clock API. > > Add CSI2 clock struct for handling it with clock API when TI PM is disabled. > + .clkdm = { > + .name = "cam_clkdm" > + }, Sorry to make you resend these again, but these should indeed be on one line, the way you had it before, to conform with the rest of the file. - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html