From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> To: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Subject: Re: [PATCH 2/3] ARM: dts: sun5i: Add sun5i-q8-common.dtsi Date: Thu, 10 Sep 2015 17:05:39 +0200 [thread overview] Message-ID: <55F19C43.2000601@redhat.com> (raw) In-Reply-To: <20150909140156.GC9885@lukather> Hi, On 09-09-15 16:01, Maxime Ripard wrote: > On Tue, Sep 08, 2015 at 11:26:51PM +0200, Hans de Goede wrote: >> Hi, >> >> On 09/08/2015 04:33 PM, Maxime Ripard wrote: >>> On Sat, Sep 05, 2015 at 04:55:52PM +0200, Hans de Goede wrote: >>>> This is the sun5i / a13 version of sun8i-q8-common.dtsi for use in dts >>>> files for a13 q8 based tablets. Compared to sun8i this uses uart1 for the >>>> serial console, and PG0 for card-detect for mmc0. >>>> >>>> This also adds pmic and otg support, which both use the same config on >>>> all known q8 a13 devices. This is not present in sun5i-q8-common.dtsi >>>> because pmic / otg support for sun8i has not yet been merged. >>>> >>>> Signed-off-by: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> >>> >>> Are we going to have any user but the one you posted in the patch 3 >>> planned? >> >> Depends on what I can get my hands on / what is in the post atm. >> >> One difference in fex files of different 18 variants is how the usb wifi is >> powered, either via a gpio or via ldo3. Also there are differences in which >> accelerometer is used, so eventually I expect there to be several users >> of this dtsi. > > But no difference in the SoC used (A13 vs A10s)? > > I'm not sure having 18 DTs for a single board name is going to be > really helpful. Sorry I meant to write q8 not 18 :) >It would be something to be supported by some quirks > mechanism, instead of having to require the user to pick the right one. Agreed, which is why I added a generic sun5i-a13-q8-tablet.dts, which can then later be extended with quirks and such to get all the different variants supported. Which I guess means that the contents of sun5i-q8-common.dtsi can be folded into sun5i-a13-q8-tablet.dts, otoh we may have some special cases which do require a separate dts and having sun5i-q8-common.dtsi follows the same scheme as used for the a23 / a33 tablets. In the future we should probably also add sun8i-a23-q8-tablet.dts and sun8i-a33-q8-tablet.dts files and make the existing ones just empty files with a single #include for compatibility. If possible I would like to at least have you queue up the default sun5i-a13-q8-tablet.dts so that we've a dts file for a13 based tablets which at least gets the basics going. Either in its current form, or with it and sun5i-q8-common.dtsi squashed together. If you want a squashed version let me know and I'll do a resend. Are you coming to ELCE ? I think the both of us should go to: http://lceeu2015.sched.org/event/999c45c7fe78824a88269f35539c7e5c#.VfGcCLPYqlM (Device Tree NG BoF) and try to hash out a solution for this there, the quirks / variants stuff sounds good, but I wonder how far along that is ? Regards, Hans
WARNING: multiple messages have this Message-ID (diff)
From: hdegoede@redhat.com (Hans de Goede) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 2/3] ARM: dts: sun5i: Add sun5i-q8-common.dtsi Date: Thu, 10 Sep 2015 17:05:39 +0200 [thread overview] Message-ID: <55F19C43.2000601@redhat.com> (raw) In-Reply-To: <20150909140156.GC9885@lukather> Hi, On 09-09-15 16:01, Maxime Ripard wrote: > On Tue, Sep 08, 2015 at 11:26:51PM +0200, Hans de Goede wrote: >> Hi, >> >> On 09/08/2015 04:33 PM, Maxime Ripard wrote: >>> On Sat, Sep 05, 2015 at 04:55:52PM +0200, Hans de Goede wrote: >>>> This is the sun5i / a13 version of sun8i-q8-common.dtsi for use in dts >>>> files for a13 q8 based tablets. Compared to sun8i this uses uart1 for the >>>> serial console, and PG0 for card-detect for mmc0. >>>> >>>> This also adds pmic and otg support, which both use the same config on >>>> all known q8 a13 devices. This is not present in sun5i-q8-common.dtsi >>>> because pmic / otg support for sun8i has not yet been merged. >>>> >>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com> >>> >>> Are we going to have any user but the one you posted in the patch 3 >>> planned? >> >> Depends on what I can get my hands on / what is in the post atm. >> >> One difference in fex files of different 18 variants is how the usb wifi is >> powered, either via a gpio or via ldo3. Also there are differences in which >> accelerometer is used, so eventually I expect there to be several users >> of this dtsi. > > But no difference in the SoC used (A13 vs A10s)? > > I'm not sure having 18 DTs for a single board name is going to be > really helpful. Sorry I meant to write q8 not 18 :) >It would be something to be supported by some quirks > mechanism, instead of having to require the user to pick the right one. Agreed, which is why I added a generic sun5i-a13-q8-tablet.dts, which can then later be extended with quirks and such to get all the different variants supported. Which I guess means that the contents of sun5i-q8-common.dtsi can be folded into sun5i-a13-q8-tablet.dts, otoh we may have some special cases which do require a separate dts and having sun5i-q8-common.dtsi follows the same scheme as used for the a23 / a33 tablets. In the future we should probably also add sun8i-a23-q8-tablet.dts and sun8i-a33-q8-tablet.dts files and make the existing ones just empty files with a single #include for compatibility. If possible I would like to at least have you queue up the default sun5i-a13-q8-tablet.dts so that we've a dts file for a13 based tablets which at least gets the basics going. Either in its current form, or with it and sun5i-q8-common.dtsi squashed together. If you want a squashed version let me know and I'll do a resend. Are you coming to ELCE ? I think the both of us should go to: http://lceeu2015.sched.org/event/999c45c7fe78824a88269f35539c7e5c#.VfGcCLPYqlM (Device Tree NG BoF) and try to hash out a solution for this there, the quirks / variants stuff sounds good, but I wonder how far along that is ? Regards, Hans
next prev parent reply other threads:[~2015-09-10 15:05 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-09-05 14:55 [PATCH 1/3] ARM: dts: sunxi: Add sun?i-q8-common.dtsi files for q8 tablets Hans de Goede 2015-09-05 14:55 ` Hans de Goede [not found] ` <1441464953-26351-1-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-09-05 14:55 ` [PATCH 2/3] ARM: dts: sun5i: Add sun5i-q8-common.dtsi Hans de Goede 2015-09-05 14:55 ` Hans de Goede [not found] ` <1441464953-26351-2-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-09-08 14:33 ` Maxime Ripard 2015-09-08 14:33 ` Maxime Ripard 2015-09-08 21:26 ` Hans de Goede 2015-09-08 21:26 ` Hans de Goede [not found] ` <55EF529B.80401-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-09-09 14:01 ` Maxime Ripard 2015-09-09 14:01 ` Maxime Ripard 2015-09-10 15:05 ` Hans de Goede [this message] 2015-09-10 15:05 ` Hans de Goede [not found] ` <55F19C43.2000601-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-09-18 19:53 ` Maxime Ripard 2015-09-18 19:53 ` Maxime Ripard 2015-09-18 19:38 ` Maxime Ripard 2015-09-18 19:38 ` Maxime Ripard 2015-09-05 14:55 ` [PATCH 3/3] ARM: dts: sun5i: Add dts file for q8 formfactor a13 based tablets Hans de Goede 2015-09-05 14:55 ` Hans de Goede [not found] ` <1441464953-26351-3-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-09-18 19:41 ` Maxime Ripard 2015-09-18 19:41 ` Maxime Ripard 2015-09-08 14:31 ` [PATCH 1/3] ARM: dts: sunxi: Add sun?i-q8-common.dtsi files for q8 tablets Maxime Ripard 2015-09-08 14:31 ` 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=55F19C43.2000601@redhat.com \ --to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \ --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@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.