From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: Re: [PATCH 2/3] ARM: dts: axp152: Add a dtsi file for the axp152 pmic Date: Fri, 26 Jun 2015 10:22:26 +0200 Message-ID: <558D0BC2.10004@redhat.com> References: <1435088503-20776-1-git-send-email-hdegoede@redhat.com> <1435088503-20776-3-git-send-email-hdegoede@redhat.com> <20150625174027.GF2266@lukather> Reply-To: hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Return-path: In-Reply-To: <20150625174027.GF2266@lukather> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Maxime Ripard Cc: Lee Jones , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Michal Suchanek List-Id: devicetree@vger.kernel.org Hi, On 25-06-15 19:40, Maxime Ripard wrote: > Hi, > > On Tue, Jun 23, 2015 at 09:41:42PM +0200, Hans de Goede wrote: >> From: Michal Suchanek >> >> Add a dtsi file for the axp152 pmic, this mirrors the way things are >> handled for the axp202 pmic. >> >> Signed-off-by: Hans de Goede >> --- >> arch/arm/boot/dts/axp152.dtsi | 49 +++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 49 insertions(+) >> create mode 100644 arch/arm/boot/dts/axp152.dtsi > > Unfortunately, Mark expressed clearly that he didn't want such files. That is not how I remember the discussion, he said that he did not consider them useful, we explained why we do consider them useful for the sunxi case and then things went quiet. And we do already have an axp209.dtsi, and I find having that useful for enabling the regulators on axp209 using boards. Ultimately it is your call, but I really think we should be consistent and add an axp152.dtsi to mirror how we do things with the axp209. Regards, Hans