> -----Original Message----- > From: Andrei Emeltchenko [mailto:[email protected]] > Sent: Friday, November 12, 2010 6:50 PM > To: G, Manjunath Kondaiah > Cc: [email protected]; [email protected]; > [email protected]; [email protected] > Subject: Re: [PATCHv4 0/4] OMAP: Serial: remove magic and diplicates > > On Fri, Nov 12, 2010 at 3:17 PM, G, Manjunath Kondaiah > <[email protected]> wrote: > > > > > >> -----Original Message----- > >> From: [email protected] > >> [mailto:[email protected]] On Behalf Of > >> Emeltchenko Andrei > >> Sent: Friday, November 12, 2010 6:34 PM > >> To: [email protected]; [email protected]; > >> [email protected]; [email protected] > >> Subject: [PATCHv4 0/4] OMAP: Serial: remove magic and diplicates > > > > s/diplicates/duplicates > > > >> > >> From: Andrei Emeltchenko <[email protected]> > >> > >> Add MDR1 Serial register definition to serial_reg and remove > >> duplicated > >> code, remove magic MDR1 register numbers from serial drivers. > >> > >> *v2 add OMAP keyword to MDR1 definition > >> *v3 remove magic from serial drivers > >> *v4 taking comments from Manjunath and Govindraj: typos, CC > >> > >> Andrei Emeltchenko (4): > >> OMAP: Serial: Define OMAP uart MDR1 register definitions > >> OMAP1: Serial: remove magic numbers for MDR1 reg > >> OMAP2: Serial: remove magic numbers for MDR1 reg > >> OMAP: Serial: remove redefined MDR1 register definitions > >> > >> arch/arm/mach-omap1/serial.c | 6 ++++-- > >> arch/arm/mach-omap2/serial.c | 15 > +++++++++------ > >> arch/arm/plat-omap/include/plat/omap-serial.h | 3 --- > >> drivers/serial/omap-serial.c | 6 +++--- > >> include/linux/serial_reg.h | 12 ++++++++++++ > >> 5 files changed, 28 insertions(+), 14 deletions(-) > > > > Do you think patch series is required for these changes? How about > > merging with single patch? > > I can merge, sometimes somebody need patches to be split,
why? can you give example? Please read: http://lwn.net/Articles/139918/ "if you make a single change to numerous files, group those changes into a single patch. Thus a single logical change is contained within a single patch." -Manjunath-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
