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
| ||
|
Date: Tue, 2 Jun 2015 10:59:28 +0200 From: Linus Walleij <linus.walleij@...aro.org> To: Bintian Wang <bintian.wang@...wei.com>, Russell King - ARM Linux <linux@....linux.org.uk> Cc: "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Catalin Marinas <catalin.marinas@....com>, Will Deacon <will.deacon@....com>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, Rob Herring <robh+dt@...nel.org>, Paweł Moll <pawel.moll@....com>, Mark Rutland <mark.rutland@....com>, "ijc+devicetree@...lion.org.uk" <ijc+devicetree@...lion.org.uk>, Kumar Gala <galak@...eaurora.org>, Kevin Hilman <khilman@...aro.org>, Mike Turquette <mturquette@...aro.org>, Rob Herring <rob.herring@...aro.org>, Zhangfei Gao <zhangfei.gao@...aro.org>, Haojian Zhuang <haojian.zhuang@...aro.org>, Xu Wei <xuwei5@...ilicon.com>, Jaehoon Chung <jh80.chung@...sung.com>, Olof Johansson <olof@...om.net>, yanhaifeng@...il.com, Stephen Boyd <sboyd@...eaurora.org>, xuejiancheng@...wei.com, sledge.yanwei@...wei.com, Tomeu Vizoso <tomeu.vizoso@...labora.com>, Guodong Xu <guodong.xu@...aro.org>, jorge.ramirez-ortiz@...aro.org, Tyler Baker <tyler.baker@...aro.org>, Kevin Hilman <khilman@...nel.org>, Paul Bolle <pebolle@...cali.nl>, Arnd Bergmann <arnd@...db.de>, Marc Zyngier <marc.zyngier@....com>, xuyiping@...ilicon.com, wangbinghui@...ilicon.com, zhenwei.wang@...ilicon.com, victor.lixin@...ilicon.com, puck.chen@...ilicon.com, dan.zhao@...ilicon.com, Xinwei Hu <huxinwei@...wei.com>, z.liuxinliang@...wei.com, Yunlei He <heyunlei@...wei.com>, XinWei Kong <kong.kongxinwei@...ilicon.com>, wangbintian@...il.com, w.f@...wei.com, liguozhu@...ilicon.com Subject: Re: [PATCH v9 4/6] Documentation: DT: PL011: hi6220: add compatible string for Hisilicon designed UART On Sat, May 30, 2015 at 3:50 AM, Bintian Wang <bintian.wang@...wei.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 patch. > future optimisations or workarounds works. > > Signed-off-by: Bintian Wang <bintian.wang@...wei.com> > Suggested-by: Mark Rutland <mark.rutland@....com> Maybe I missed out on the earlier conversation, but do you mean that the PrimeCell ID has not been properly set up to something unique in this HiSilicon version of the PL011 block? Even if so: do not override the compatible string like this, that is not the PrimeCell style. Define an 8 bit vendor ID (like tha ASCII for 'H' 0x48 or whatever) and encode it for these variants, if the hardware is just using the ARM default PrimeCell ID, override it in the device tree like Broadcom are doing in arch/arm/boot/dts/bcm2835.dtsi: arm,primecell-periphid = <0x00241011>; Maybe yours would be: arm,primecell-periphid = <0x00048011>; For a first HiSilicon variant, then do some <include/linux/amba/bus.h>: enum amba_vendor { AMBA_VENDOR_ARM = 0x41, + AMBA_VENDOR_HISILICON = 0x48, Then patch drivers/tty/serial/amba_pl011.c to add vendor_hisilicon and a match table for 0x00048011 just like everyone else. Yours, Linus Walleij -- 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/
Powered by blists - more mailing lists