On Thu, 2011-03-10 at 02:52 -0600, Premi, Sanjeev wrote: > > -----Original Message----- > > From: K, Mythri P > > Sent: Thursday, March 10, 2011 10:32 AM > > To: Premi, Sanjeev > > Cc: linux-omap@xxxxxxxxxxxxxxx; Valkeinen, Tomi > > Subject: Re: [PATCH v4 4/9] OMAP4 : DSS2 : HDMI: HDMI driver header file > > addition > > > > [snip]...[snip] > > > > > > > [sp] Any specific reason to keep header file in separate patch? > > > > > When i had posted it earlier there were several compliants that the > > patch is huge, > > if i include header file with the corresponding c file , and hence > > difficult to review. So it just to make > > reviewers job easy. > > [sp] Final say on the issue rests with the maintainer. But it I found it > difficult to review the .c file with when looking for structure > definitions, members etc. > > If they were together, we wouldn't need this in a separate patch: > [PATCH v4 7/9] OMAP4 : DSS : HDMI: Call to HDMI module init to... It's not a clear thing, but I find it easier to review the patches, if new large files are added as separate patches. But it's true that that introduces the need to do some cross checking between patches, so it's a compromise. Tomi -- 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