linux-kernel - Re: [PATCH v2 3/6] clk: hi6220: Document devicetree bindings for hi6220 clock

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]

Message-ID: <5065055.RUNnjWsHWj@wuerfel>
Date:	Tue, 14 Apr 2015 15:20:28 +0200
From:	Arnd Bergmann <arnd@...db.de>
To:	linux-arm-kernel@...ts.infradead.org
Cc:	Bintian <bintian.wang@...wei.com>, mark.rutland@....com,
	dan.zhao@...ilicon.com, btw@...l.itp.ac.cn,
	catalin.marinas@....com, wangbinghui@...ilicon.com,
	will.deacon@....com, huxinwei@...wei.com,
	haojian.zhuang@...aro.org, yanhaifeng@...il.com,
	rob.herring@...aro.org, mturquette@...aro.org,
	xuwei5@...ilicon.com, jh80.chung@...sung.com,
	sledge.yanwei@...wei.com, kong.kongxinwei@...ilicon.com,
	heyunlei@...wei.com, w.f@...wei.com, zhangfei.gao@...aro.org,
	z.liuxinliang@...wei.com, devicetree@...r.kernel.org,
	khilman@...aro.org, pawel.moll@....com,
	ijc+devicetree@...lion.org.uk, puck.chen@...ilicon.com,
	xuejiancheng@...wei.com, robh+dt@...nel.org,
	linux@....linux.org.uk, tyler.baker@...aro.org,
	zhenwei.wang@...ilicon.com, guodong.xu@...aro.org,
	victor.lixin@...ilicon.com, sboyd@...eaurora.org,
	linux-kernel@...r.kernel.org, tomeu.vizoso@...labora.com,
	galak@...eaurora.org, olof@...om.net,
	jorge.ramirez-ortiz@...aro.org, xuyiping@...ilicon.com,
	liguozhu@...ilicon.com
Subject: Re: [PATCH v2 3/6] clk: hi6220: Document devicetree bindings for hi6220 clock

On Tuesday 14 April 2015 20:37:11 Bintian wrote:
> It's really a smart fix!
> 
> For the four system controllers, how about change the following strings:
> +       - "hisilicon,aoctrl"
> +       - "hisilicon,sysctrl"
> +       - "hisilicon,mediactrl"
> +       - "hisilicon,pmctrl"
> 
> to
> +       - "hisilicon,hi6220-aoctrl"
> +       - "hisilicon,hi6220-sysctrl"
> +       - "hisilicon,hi6220-mediactrl"
> +       - "hisilicon,hi6220-pmctrl"

Yes, that works.

> and I also use "hisilicon,hi6220-xxxx" for hi6220 clk driver directly ?

Seems fine to me, though I'd have to look at the bigger picture to see
any potential problems. At the moment, I don't see any.

	Arnd
--
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/