[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130604164555.GF31367@sirena.org.uk>
Date:	Tue, 4 Jun 2013 17:45:55 +0100
From:	Mark Brown <broonie@...nel.org>
To:	Guodong Xu <guodong.xu@...aro.org>
Cc:	sameo@...ux.intel.com, lgirdwood@...il.com,
	linux-kernel@...r.kernel.org, patches@...aro.org
Subject: Re: [PATCH 3/3] ARM: dts: add dedicated dtsi for hi6421
On Tue, Jun 04, 2013 at 10:28:43PM +0800, Guodong Xu wrote:
> +	ldo1: ldo@21 {
> +		compatible = "hisilicon,hi6421-ldo";
> +		regulator-name = "LDO1";
> +		regulator-min-microvolt = <1700000>;
> +		regulator-max-microvolt = <2000000>;
This all looks very board specific.  There's no point in names like
"LDO1" really - the goal for names is to provide something which matches
the schematics to aid diagnostics.
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists