From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752296AbdI2IdH (ORCPT ); Fri, 29 Sep 2017 04:33:07 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:42621 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751436AbdI2IdD (ORCPT ); Fri, 29 Sep 2017 04:33:03 -0400 Date: Fri, 29 Sep 2017 10:32:51 +0200 From: Maxime Ripard To: Chen-Yu Tsai Cc: Lee Jones , Mark Brown , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Quentin Schulz Subject: Re: [PATCH 4/7] ARM: dts: sunxi: Add dtsi for AXP81x PMIC Message-ID: <20170929083251.3akrjkujbqeqbh2x@flea> References: <20170929032514.31727-1-wens@csie.org> <20170929032514.31727-5-wens@csie.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="xaghazfqgpnxqpyg" Content-Disposition: inline In-Reply-To: <20170929032514.31727-5-wens@csie.org> User-Agent: NeoMutt/20170914 (1.9.0) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --xaghazfqgpnxqpyg Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Sep 29, 2017 at 03:25:11AM +0000, Chen-Yu Tsai wrote: > The AXP81x family of PMIC is used with the Allwinner A83T and H8 SoCs. > This includes the AXP813 and AXP818. There is no discernible difference > except the labeling. The AXP813 is paired with the A83T, while the > AXP818 is paired with the H8. >=20 > This patch adds a dtsi file for all the common bindings for these two > PMICs. Currently this is just listing all the regulator nodes. The > regulators are initialized based on their device node names. >=20 > In the future this would be expanded to include power supplies and > GPIO controllers. >=20 > Signed-off-by: Chen-Yu Tsai > --- > arch/arm/boot/dts/axp81x.dtsi | 139 ++++++++++++++++++++++++++++++++++++= ++++++ > 1 file changed, 139 insertions(+) > create mode 100644 arch/arm/boot/dts/axp81x.dtsi >=20 > diff --git a/arch/arm/boot/dts/axp81x.dtsi b/arch/arm/boot/dts/axp81x.dtsi > new file mode 100644 > index 000000000000..73b761f850c5 > --- /dev/null > +++ b/arch/arm/boot/dts/axp81x.dtsi > @@ -0,0 +1,139 @@ > +/* > + * Copyright 2017 Chen-Yu Tsai > + * > + * Chen-Yu Tsai > + * > + * This file is dual-licensed: you can use it either under the terms > + * of the GPL or the X11 license, at your option. Note that this dual > + * licensing only applies to this file, and not this project as a > + * whole. > + * > + * a) This file is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as > + * published by the Free Software Foundation; either version 2 of the > + * License, or (at your option) any later version. > + * > + * This file is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * Or, alternatively, > + * > + * b) Permission is hereby granted, free of charge, to any person > + * obtaining a copy of this software and associated documentation > + * files (the "Software"), to deal in the Software without > + * restriction, including without limitation the rights to use, > + * copy, modify, merge, publish, distribute, sublicense, and/or > + * sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following > + * conditions: > + * > + * The above copyright notice and this permission notice shall be > + * included in all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES > + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND > + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT > + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, > + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR > + * OTHER DEALINGS IN THE SOFTWARE. > + */ > + > +/* AXP813/818 Integrated Power Management Chip */ > + > +&axp81x { > + interrupt-controller; > + #interrupt-cells =3D <1>; > + > + regulators { > + /* Default work frequency for buck regulators */ > + x-powers,dcdc-freq =3D <3000>; > + > + reg_dcdc1: dcdc1 { > + }; Is there any reason not to add some default regulator names here? Thanks! Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --xaghazfqgpnxqpyg Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBAgAGBQJZzgUzAAoJEBx+YmzsjxAg4eEP/AlLCfAU2LzHJVjAZByPuQbv BA93NAMunO8XegdL2hOLZn6fzykRbKDsEfVOCLo9x4rihKTiHhUx3lKTkSXYikLR bCTJwPaYAiRVlQeWgNL3BaDAGAsYOvNrQhTUHOw34vpXlYI9O6QSoec3mnTY1feE xD7jMDvNDkPN9YeMd2Slazljum56e7bCFlYdNSfYsjoac0CPdxbVjgT3IejfVRYh ZXe8XvVGApg/GsShbBtvizqeuOdnPlkqXqs16y821RHA97Mk6X2kSY4KJlSLe0wB LE0TtQL/Hev0OhlncR+HbeNYrlrkeiR6T+Mlv6veFsbBIbRXI8otMhqVISpPTiLF XP0KXcl1j0vbS9qjwt5jKCAXzrK+buiGvrXWk293Lg4bf4lYX4WfxzWOL3GXb1/4 mZVG7C6MNFw/jgfJqQb6EdIuSmR51XjzDmJoc211JstxUV6jRl/pslotytxpW3TB v8OZiPsT4q+jgen5lBrH3xKgK78MlPL3R/QmU67MRBx9aPNfx+Dx8VA3bwuNJmfC 93TJKPcjDPIFGlVXvsRqjba0HkhtXeoEHvQJqgik4xaN/sJCNqGBjblJF1zFp+GE 636nuysB9620nvWJ8Rxd9lMK0sOEfNN5/7Ip5CM0PF476MfYqUtS2ogtZxBi4IzP NJqba/JiSrB/ezcXvnGH =DU+b -----END PGP SIGNATURE----- --xaghazfqgpnxqpyg--