From: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> To: Carlo Caione <carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> Cc: linux-arm-kernel <linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>, Hans De Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Emilio Lopez <emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>, wens Tsai <wens-jdAy2FN1RRM@public.gmane.org>, sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, Dmitry Torokhov <dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Liam Girdwood <lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>, boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Subject: Re: Re: [PATCH v5 8/8] ARM: sun7i/sun4i: dt: Add AXP209 support to various boards Date: Fri, 9 May 2014 20:06:34 -0500 [thread overview] Message-ID: <20140510010634.GG7047@lukather> (raw) In-Reply-To: <CAOQ7t2ay7UNiCZyUw4RrQ+4O1CYUht32U_OSYkbVWRLUq8_hZw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> [-- Attachment #1: Type: text/plain, Size: 2393 bytes --] On Thu, May 08, 2014 at 10:17:16AM +0200, Carlo Caione wrote: > On Thu, May 8, 2014 at 4:53 AM, Maxime Ripard > <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote: > > <snip> > > > What I mean is that no information is still better than wrong > > informations. If you don't have the schematics, then don't do anything > > on this boards. Hopefully, someone with more infos on this will know > > what to do. > > I partially agree here. Not specifying the voltage range for the > regulators but only keeping them always-on from a practical point of > view means that we do nothing with those regulators (see also > https://www.mail-archive.com/linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org/msg09532.html > ) But we don't even know if they are used on these boards or not in the first place.. > > Moreover, I have the feeling that you just copy pasted all the > > informations on this patch. The first example I'm seeing is the > > cubieboard, for which we do have the schematics, and this patch says > > that ldo4 is used for HDMI, and that is required to stays always on, > > while LDO4 doesn't seem to be used at all, and that HDMI takes > > directly its input source from DC-5V. > > The names for the regulators are taken from the original Allwinner > driver and (of course) they can have nothing to do with the function > they have in the board. I agree that this could be misleading so I'll > change them with more general names. Concerning LDO4 that is always > on, it was off until this v4, but I changed it back to have one common > DT representation among all the boards. I'm sorry, you did a great job on this PMIC driver, but this is just wrong. I'm completely fine with the names. What I'm not fine with is all this "one common DT representation across all boards". Pretty much all the boards are wired differently when it comes to regulators. Why do you want to just do some copy pasting without any checking on the schematics side if not just for the joy of duplicating code? > > I guess you need some more refining on this patch. > > Ok, I'll submit a v6 only for this patch. Please, please, *please* make sure that the informations you put in there are actually accurate. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@free-electrons.com (Maxime Ripard) To: linux-arm-kernel@lists.infradead.org Subject: [linux-sunxi] Re: [PATCH v5 8/8] ARM: sun7i/sun4i: dt: Add AXP209 support to various boards Date: Fri, 9 May 2014 20:06:34 -0500 [thread overview] Message-ID: <20140510010634.GG7047@lukather> (raw) In-Reply-To: <CAOQ7t2ay7UNiCZyUw4RrQ+4O1CYUht32U_OSYkbVWRLUq8_hZw@mail.gmail.com> On Thu, May 08, 2014 at 10:17:16AM +0200, Carlo Caione wrote: > On Thu, May 8, 2014 at 4:53 AM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > <snip> > > > What I mean is that no information is still better than wrong > > informations. If you don't have the schematics, then don't do anything > > on this boards. Hopefully, someone with more infos on this will know > > what to do. > > I partially agree here. Not specifying the voltage range for the > regulators but only keeping them always-on from a practical point of > view means that we do nothing with those regulators (see also > https://www.mail-archive.com/linux-input at vger.kernel.org/msg09532.html > ) But we don't even know if they are used on these boards or not in the first place.. > > Moreover, I have the feeling that you just copy pasted all the > > informations on this patch. The first example I'm seeing is the > > cubieboard, for which we do have the schematics, and this patch says > > that ldo4 is used for HDMI, and that is required to stays always on, > > while LDO4 doesn't seem to be used at all, and that HDMI takes > > directly its input source from DC-5V. > > The names for the regulators are taken from the original Allwinner > driver and (of course) they can have nothing to do with the function > they have in the board. I agree that this could be misleading so I'll > change them with more general names. Concerning LDO4 that is always > on, it was off until this v4, but I changed it back to have one common > DT representation among all the boards. I'm sorry, you did a great job on this PMIC driver, but this is just wrong. I'm completely fine with the names. What I'm not fine with is all this "one common DT representation across all boards". Pretty much all the boards are wired differently when it comes to regulators. Why do you want to just do some copy pasting without any checking on the schematics side if not just for the joy of duplicating code? > > I guess you need some more refining on this patch. > > Ok, I'll submit a v6 only for this patch. Please, please, *please* make sure that the informations you put in there are actually accurate. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140509/abd40966/attachment.sig>
next prev parent reply other threads:[~2014-05-10 1:06 UTC|newest] Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-05-01 12:29 [PATCH v5 0/8] mfd: AXP20x: Add support for AXP202 and AXP209 Carlo Caione 2014-05-01 12:29 ` Carlo Caione [not found] ` <1398947374-3115-1-git-send-email-carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> 2014-05-01 12:29 ` [PATCH v5 1/8] mfd: AXP20x: Add mfd driver for AXP20x PMIC Carlo Caione 2014-05-01 12:29 ` Carlo Caione [not found] ` <1398947374-3115-2-git-send-email-carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> 2014-05-02 8:15 ` Lee Jones 2014-05-02 8:15 ` Lee Jones 2014-05-02 8:27 ` Carlo Caione 2014-05-02 8:27 ` [linux-sunxi] " Carlo Caione 2014-05-06 7:14 ` Lee Jones 2014-05-06 7:14 ` Lee Jones 2014-05-03 1:03 ` Maxime Ripard 2014-05-03 1:03 ` Maxime Ripard 2014-05-01 12:29 ` [PATCH v5 2/8] dt-bindings: add vendor-prefix for X-Powers Carlo Caione 2014-05-01 12:29 ` Carlo Caione 2014-05-02 5:24 ` Michal Simek 2014-05-02 5:24 ` Michal Simek [not found] ` <53632C16.6070105-pSz03upnqPeHXe+LvDLADg@public.gmane.org> 2014-05-02 7:24 ` Carlo Caione 2014-05-01 12:29 ` [PATCH v5 3/8] mfd: AXP20x: Add bindings documentation Carlo Caione 2014-05-01 12:29 ` Carlo Caione [not found] ` <1398947374-3115-4-git-send-email-carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> 2014-05-03 1:02 ` Maxime Ripard 2014-05-03 1:02 ` Maxime Ripard 2014-05-01 12:29 ` [PATCH v5 4/8] input: misc: Add driver for AXP20x Power Enable Key Carlo Caione 2014-05-01 12:29 ` Carlo Caione [not found] ` <1398947374-3115-5-git-send-email-carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> 2014-05-03 1:06 ` Maxime Ripard 2014-05-03 1:06 ` Maxime Ripard 2014-05-19 5:57 ` Dmitry Torokhov 2014-05-19 5:57 ` Dmitry Torokhov [not found] ` <20140519055745.GL29386-WlK9ik9hQGAhIp7JRqBPierSzoNAToWh@public.gmane.org> 2014-05-19 7:18 ` Carlo Caione 2014-05-19 7:18 ` [linux-sunxi] " Carlo Caione [not found] ` <CAOQ7t2bPHnjfpeqQjMsYJs--qpd1_1RvBEFXztTDgnf5yW=Yxg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-05-19 9:15 ` Lee Jones 2014-05-19 9:15 ` [linux-sunxi] " Lee Jones 2014-05-19 9:39 ` Carlo Caione 2014-05-19 9:39 ` [linux-sunxi] " Carlo Caione 2014-05-19 13:28 ` Lee Jones 2014-05-19 13:28 ` Lee Jones 2014-05-19 7:23 ` Lee Jones 2014-05-19 7:23 ` Lee Jones 2014-05-01 12:29 ` [PATCH v5 5/8] input: misc: Add ABI docs for AXP20x PEK Carlo Caione 2014-05-01 12:29 ` Carlo Caione 2014-05-01 12:29 ` [PATCH v5 6/8] ARM: sunxi: Add AXP20x support in defconfig Carlo Caione 2014-05-01 12:29 ` Carlo Caione 2014-05-01 12:29 ` [PATCH v5 7/8] ARM: sunxi: Add AXP20x support multi_v7_defconfig Carlo Caione 2014-05-01 12:29 ` Carlo Caione 2014-05-01 12:29 ` [PATCH v5 8/8] ARM: sun7i/sun4i: dt: Add AXP209 support to various boards Carlo Caione 2014-05-01 12:29 ` Carlo Caione [not found] ` <1398947374-3115-9-git-send-email-carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org> 2014-05-03 1:09 ` Maxime Ripard 2014-05-03 1:09 ` Maxime Ripard 2014-05-03 12:21 ` Carlo Caione 2014-05-03 12:21 ` [linux-sunxi] " Carlo Caione 2014-05-05 22:51 ` Maxime Ripard 2014-05-05 22:51 ` Maxime Ripard 2014-05-06 7:38 ` Carlo Caione 2014-05-06 7:38 ` Carlo Caione [not found] ` <CAOQ7t2bg5jkeruVnO+9cXMxxh+SL+ANGz3R0ssSMwrHewRKyYQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-05-08 2:53 ` Maxime Ripard 2014-05-08 2:53 ` [linux-sunxi] " Maxime Ripard 2014-05-08 8:17 ` Carlo Caione 2014-05-08 8:17 ` [linux-sunxi] " Carlo Caione [not found] ` <CAOQ7t2ay7UNiCZyUw4RrQ+4O1CYUht32U_OSYkbVWRLUq8_hZw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-05-10 1:06 ` Maxime Ripard [this message] 2014-05-10 1:06 ` Maxime Ripard
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140510010634.GG7047@lukather \ --to=maxime.ripard-wi1+55scjutkeb57/3fjtnbpr1lh4cv8@public.gmane.org \ --cc=boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \ --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org \ --cc=dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org \ --cc=hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ --cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \ --cc=sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \ --cc=wens-jdAy2FN1RRM@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.