linux-kernel - Re: [PATCH v6 5/5] hisilicon/dts: Add hi655x pmic dts node

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: <20160120130813.GD6588@sirena.org.uk>
Date:	Wed, 20 Jan 2016 13:08:13 +0000
From:	Mark Brown <broonie@...nel.org>
To:	Chen Feng <puck.chen@...ilicon.com>
Cc:	lee.jones@...aro.org, lgirdwood@...il.com,
	linux-kernel@...r.kernel.org, yudongbin@...ilicon.com,
	saberlily.xia@...ilicon.com, suzhuangluan@...ilicon.com,
	kong.kongxinwei@...ilicon.com, xuyiping@...ilicon.com,
	z.liuxinliang@...ilicon.com, weidong2@...ilicon.com,
	w.f@...wei.com, puck.chen@...mail.com, shimingxing@...ilicon.com,
	oliver.fu@...ilicon.com, albert.lubing@...ilicon.com,
	chenxiang9@...wei.com, liuzixing@...wei.com,
	haojian.zhuang@...aro.org, qijiwen@...ilicon.com,
	peter.panshilin@...ilicon.com, dan.zhao@...ilicon.com,
	linuxarm@...wei.com, dev@...ts.96boards.org
Subject: Re: [PATCH v6 5/5] hisilicon/dts: Add hi655x pmic dts node

On Tue, Jan 19, 2016 at 02:32:04PM +0800, Chen Feng wrote:

> index 82d2488..6de9881 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi

> +		regulators {
> +			ldo2: ldo2@a21 {
> +				regulator-compatible = "LDO2";

Why are you using the legacy regulator-compatible property?  No new
bindings should use this.

> +				regulator-min-microvolt = <2500000>;
> +				regulator-max-microvolt = <3200000>;

This is broken as it misunderstands the purpose of specifying
constraints.  The constraints are there to say what the safe and
supported configuration is on a given board, it is not possible to
provide this information safely in a general include that is used by all
systems using the PMIC.  Specifying the maximum voltage range for the
regulators is almost guaranteed to result in at least some
configurations being enabled which will not work, in the worst case this
may include configurations which could physically damage the system.

In general it is very unusual to include the regulators in a .dtsi since
essentially all the configuration for them should be board specific.

Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)