> -----Original Message----- > From: Andrei Emeltchenko [mailto:andrei.emeltchenko.n...@gmail.com] > Sent: Friday, November 12, 2010 6:50 PM > To: G, Manjunath Kondaiah > Cc: linux-omap@vger.kernel.org; linux-ker...@vger.kernel.org; > linux-...@lists.arm.linux.org.uk; linux-ser...@vger.kernel.org > Subject: Re: [PATCHv4 0/4] OMAP: Serial: remove magic and diplicates > > On Fri, Nov 12, 2010 at 3:17 PM, G, Manjunath Kondaiah > <manj...@ti.com> wrote: > > > > > >> -----Original Message----- > >> From: linux-omap-ow...@vger.kernel.org > >> [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of > >> Emeltchenko Andrei > >> Sent: Friday, November 12, 2010 6:34 PM > >> To: linux-omap@vger.kernel.org; linux-ker...@vger.kernel.org; > >> linux-...@lists.arm.linux.org.uk; linux-ser...@vger.kernel.org > >> Subject: [PATCHv4 0/4] OMAP: Serial: remove magic and diplicates > > > > s/diplicates/duplicates > > > >> > >> From: Andrei Emeltchenko <andrei.emeltche...@nokia.com> > >> > >> 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html