Re: [PATCHv4 0/4] OMAP: Serial: remove magic and diplicates

Andrei Emeltchenko <[email protected]>
Newsgroups gmane.linux.serial,gmane.linux.ports.arm.omap,gmane.linux.kernel,gmane.linux.ports.arm.general
Message-ID <[email protected]>
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, merging is easy.

-- Andrei
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.