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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 28 Oct 2011 15:35:23 +0800
From:	Barry Song <21cnbao@...il.com>
To:	Baruch Siach <baruch@...s.co.il>
Cc:	torvalds@...ux-foundation.org, Rong Wang <Rong.Wang@....com>,
	Bin Shi <Bin.Shi@....com>,
	Linus Walleij <linus.walleij@...aro.org>,
	linux-kernel@...r.kernel.org, workgroup.linux@....com,
	linux-serial@...r.kernel.org, Barry Song <Baohua.Song@....com>,
	linux-arm-kernel@...ts.infradead.org, alan@...ux.intel.com
Subject: Re: [PATCH] UART: add CSR SiRFprimaII SoC on-chip uart drivers

2011/10/28 Baruch Siach <baruch@...s.co.il>:
> Hi Barry,
>
> On Thu, Oct 27, 2011 at 08:27:33PM -0700, Barry Song wrote:
>> From: Rong Wang <Rong.Wang@....com>
>>
>> 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@...aro.org>
>> Signed-off-by: Rong Wang <Rong.Wang@....com>
>> Signed-off-by: Bin Shi <Bin.Shi@....com>
>> Signed-off-by: Barry Song <Baohua.Song@....com>
>> Acked-by: Alan Cox <alan@...ux.intel.com>
>> ---
>
> [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@...s.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/