Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Bintian
Hello Russell, On 2015/6/2 19:24, Russell King - ARM Linux wrote: On Tue, Jun 02, 2015 at 06:55:20PM +0800, Bintian wrote: On 2015/6/2 16:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Russell King - ARM Linux
On Tue, Jun 02, 2015 at 06:55:20PM +0800, Bintian wrote: > On 2015/6/2 16:59, Linus Walleij wrote: > >On Sat, May 30, 2015 at 3:50 AM, Bintian Wang > >wrote: > > > >>Hisilicon does some performance enhancements based on PL011(e.g. larger > >>FIFO length), so add one compatible string

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Bintian
On 2015/6/2 16:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string "hisilicon,hi6220-uart" for That compatible string in the commit message is not

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Russell King - ARM Linux
On Tue, Jun 02, 2015 at 10:13:26AM +0100, Marc Zyngier wrote: > That feels weird. This amba_vendor enum is not under control of the DT > author, nor the kernel. This is a set of codes that are managed by a > third party (probably ARM). What if some company with a name starting > with 'H'

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Marc Zyngier
On 02/06/15 09:59, Linus Walleij wrote: > On Sat, May 30, 2015 at 3:50 AM, Bintian Wang wrote: > >> Hisilicon does some performance enhancements based on PL011(e.g. larger >> FIFO length), so add one compatible string "hisilicon,hi6220-uart" for > > That compatible string in the commit message

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Linus Walleij
On Sat, May 30, 2015 at 3:50 AM, Bintian Wang wrote: > Hisilicon does some performance enhancements based on PL011(e.g. larger > FIFO length), so add one compatible string "hisilicon,hi6220-uart" for That compatible string in the commit message is not even the same as in the patch. > future

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Marc Zyngier
On 02/06/15 09:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang bintian.w...@huawei.com wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string hisilicon,hi6220-uart for That compatible string in the

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Russell King - ARM Linux
On Tue, Jun 02, 2015 at 10:13:26AM +0100, Marc Zyngier wrote: That feels weird. This amba_vendor enum is not under control of the DT author, nor the kernel. This is a set of codes that are managed by a third party (probably ARM). What if some company with a name starting with 'H' (Hilarious

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Linus Walleij
On Sat, May 30, 2015 at 3:50 AM, Bintian Wang bintian.w...@huawei.com wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string hisilicon,hi6220-uart for That compatible string in the commit message is not even the same as in the

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Russell King - ARM Linux
On Tue, Jun 02, 2015 at 06:55:20PM +0800, Bintian wrote: On 2015/6/2 16:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang bintian.w...@huawei.com wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Bintian
On 2015/6/2 16:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang bintian.w...@huawei.com wrote: Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string hisilicon,hi6220-uart for That compatible string in the

Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-06-02 Thread Bintian
Hello Russell, On 2015/6/2 19:24, Russell King - ARM Linux wrote: On Tue, Jun 02, 2015 at 06:55:20PM +0800, Bintian wrote: On 2015/6/2 16:59, Linus Walleij wrote: On Sat, May 30, 2015 at 3:50 AM, Bintian Wang bintian.w...@huawei.com wrote: Hisilicon does some performance enhancements based

[PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-05-29 Thread Bintian Wang
Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string "hisilicon,hi6220-uart" for future optimisations or workarounds works. Signed-off-by: Bintian Wang Suggested-by: Mark Rutland ---

[PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART

2015-05-29 Thread Bintian Wang
Hisilicon does some performance enhancements based on PL011(e.g. larger FIFO length), so add one compatible string hisilicon,hi6220-uart for future optimisations or workarounds works. Signed-off-by: Bintian Wang bintian.w...@huawei.com Suggested-by: Mark Rutland mark.rutl...@arm.com ---