Re: [PATCH] UART: add CSR SiRFprimaII SoC on-chip uart drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2011/10/28 Baruch Siach <baruch@xxxxxxxxxx>:
> Hi Barry,
>
> On Thu, Oct 27, 2011 at 08:27:33PM -0700, Barry Song wrote:
>> From: Rong Wang <Rong.Wang@xxxxxxx>
>>
>> SiRFprimaII is the latest generation application processor from CSR’s
>> multi-function SoC product family.
>> The SoC support codes are in arch/arm/mach-prima2 from Linux mainline
>> 3.0.
>>
>> There are three dedicated UARTs in system. This patch adds basic driver
>> support for them.
>>
>> It has used the newest pinmux subsystem from Linus Walleij.
>>
>> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
>> Signed-off-by: Rong Wang <Rong.Wang@xxxxxxx>
>> Signed-off-by: Bin Shi <Bin.Shi@xxxxxxx>
>> Signed-off-by: Barry Song <Baohua.Song@xxxxxxx>
>> Acked-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
>> ---
>
> [snip]
>
>> +config SERIAL_SIRFSOC
>> +        tristate "SiRF SoC Platform Serial port support"
>> +        depends on ARM && ARCH_PRIMA2
>> +        select SERIAL_CORE
>> +        help
>> +          Support for the on-chip UART on the CSR SiRFprimaII series,
>> +          providing /dev/ttyS0, 1 and 2 (note, some machines may not
>> +          provide all of these ports, depending on how the serial port
>> +          pins are configured.
>
> Unbalanced parentheses.

Hi Baruch,
Thanks for reviewing! I sent this mail for Torvalds to pick seperately
since it depends on pinctl tree, and it was not gone through tty tree
since tty has no pinctrl before linus merges it.
This patch has been reviewed and acked in linux-serial, unfortunately,
missed this error.
i am not sure whether i will make Linus confused if i send a new
version at this moment. how about i send a seperate patch to fix this
after the driver is applied?

>
> baruch
>
> --
>                                                     ~. .~   Tk Open Systems
> =}------------------------------------------------ooO--U--Ooo------------{=
>   - baruch@xxxxxxxxxx - tel: +972.2.679.5364, http://www.tkos.co.il -

-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux