devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: "Emilio López" <emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>,
	"Mike Turquette"
	<mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: "Chris Ball" <chris-OsFVWbfNK3isTnJN9+BGXg@public.gmane.org>,
	"Ulf Hansson"
	<ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"David Lanzendörfer"
	<david.lanzendoerfer-Z7Kmv9EsliU@public.gmane.org>,
	"Maxime Ripard"
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
Subject: Re: [PATCH v8 00/17] ARM: sunxi: Add driver for SD/MMC hosts found on Allwinner sunxi SoCs
Date: Tue, 22 Apr 2014 14:17:30 +0200	[thread overview]
Message-ID: <53565DDA.9020202@redhat.com> (raw)
In-Reply-To: <53565B74.4000703-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>

Hi,

On 04/22/2014 02:07 PM, Emilio López wrote:
> Hi Hans,
> 
> El 22/04/14 08:01, Hans de Goede escribió:
>> Hi All,
>>
>> Here is v8 of the sunxi-mmc patch-set David Lanzendörfer and I have been
>> working on.
>>
>> The first 2 patches are depenencies which should go in through the clk tree,
>> Mike can you pick these 2 up please ?  :
>>
>> "clk: sunxi: factors: automatic reparenting support"
>> Is uncontroversial and has been favorably reviewed by various people.
> 
> I'll pick this one up for 3.16 if you're ok with that. Mike, let me know if you have any objection to the patch, but I'm pretty sure you (informally?) acked it in the past.
> 
>> "clk: sunxi: Implement MMC phase control"
>> Is somewhat more controversial as there has been lots of discussion about
>> adding a generic phase control method to the clk framework. The problem is
>> that there has been a lot of talk about such a generic phase control method
>> but not a single patch. Therefor I would like to move forwards with using
>> a platform specific method for now. I hereby promise that once we've a generic
>> method I'll write patches to convert the sunxi code to that method.
> 
> Mike proposed a RFC for handling this better, and there were talks with Allwinner people to see what did these phase controls actually mean. We should be able to get a proper solution for it during this cycle. I'd prefer to work towards that instead of merging this, as we have more than a month to get it ready still.

A month will be gone before you know it, and merging sunxi mmc support really
is long overdue, and is the last thing keeping distros from shipping with sunxi
support ootb. So I would really prefer to just get this merged as is for now,
changing it later should be easy, and merging an interim solution will make
some more time for doing the generic clk phase control method properly.

Regards,

Hans

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

  parent reply	other threads:[~2014-04-22 12:17 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-22 11:01 [PATCH v8 00/17] ARM: sunxi: Add driver for SD/MMC hosts found on Allwinner sunxi SoCs Hans de Goede
     [not found] ` <1398164497-14212-1-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-22 11:01   ` [PATCH v8 01/17] clk: sunxi: factors: automatic reparenting support Hans de Goede
2014-04-22 11:01   ` [PATCH v8 02/17] clk: sunxi: Implement MMC phase control Hans de Goede
2014-04-22 11:01   ` [PATCH v8 03/17] ARM: sunxi: Add driver for SD/MMC hosts found on Allwinner sunxi SoCs Hans de Goede
2014-04-22 11:01   ` [PATCH v8 04/17] ARM: dts: sun4i: Add mmc controller nodes Hans de Goede
2014-04-22 11:01   ` [PATCH v8 05/17] ARM: dts: sun4i: Add pin-muxing info for the mmc0 controller Hans de Goede
2014-04-22 11:01   ` [PATCH v8 06/17] ARM: dts: sun4i: Enable mmc controller on various A10 boards Hans de Goede
2014-04-22 11:01   ` [PATCH v8 07/17] ARM: dts: sun5i: Add mmc controller nodes Hans de Goede
     [not found]     ` <1398164497-14212-8-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-23  9:48       ` Maxime Ripard
2014-04-23  9:57         ` [linux-sunxi] " Hans de Goede
2014-04-22 11:01   ` [PATCH v8 08/17] ARM: dts: sun5i: Add pin-muxing info for the mmc controllers Hans de Goede
2014-04-22 11:01   ` [PATCH v8 09/17] ARM: dts: sun5i: Enable mmc controller on various A10s and A13 boards Hans de Goede
2014-04-22 11:01   ` [PATCH v8 10/17] ARM: dts: sun6i: Add mmc clocks Hans de Goede
     [not found]     ` <1398164497-14212-11-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-23  9:44       ` Maxime Ripard
2014-04-23  9:49         ` [linux-sunxi] " Hans de Goede
2014-04-22 11:01   ` [PATCH v8 11/17] ARM: dts: sun6i: Add mmc controller nodes Hans de Goede
     [not found]     ` <1398164497-14212-12-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-23  9:50       ` Maxime Ripard
2014-04-23 10:11         ` Hans de Goede
2014-04-22 11:01   ` [PATCH v8 12/17] ARM: dts: sun6i: Add pin-muxing info for the mmc controllers Hans de Goede
2014-04-22 11:01   ` [PATCH v8 13/17] ARM: dts: sun6i: Add new sun6i-a31-m9 dts file for Mele M9 Hans de Goede
2014-04-22 11:01   ` [PATCH v8 14/17] ARM: dts: sun6i: Enable the mmc controller on the m9 Hans de Goede
     [not found]     ` <1398164497-14212-15-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-23  9:51       ` Maxime Ripard
2014-04-22 11:01   ` [PATCH v8 15/17] ARM: dts: sun7i: Add mmc controller nodes Hans de Goede
2014-04-22 11:01   ` [PATCH v8 16/17] ARM: dts: sun7i: Add pin-muxing info for the mmc controllers Hans de Goede
2014-04-22 11:01   ` [PATCH v8 17/17] ARM: dts: sun7i: Enable mmc controller on various A20 boards Hans de Goede
2014-04-22 12:07   ` [PATCH v8 00/17] ARM: sunxi: Add driver for SD/MMC hosts found on Allwinner sunxi SoCs Emilio López
     [not found]     ` <53565B74.4000703-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>
2014-04-22 12:17       ` Hans de Goede [this message]
     [not found]         ` <53565DDA.9020202-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-04-23  9:40           ` Maxime Ripard
2014-04-23  9:53   ` 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=53565DDA.9020202@redhat.com \
    --to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=chris-OsFVWbfNK3isTnJN9+BGXg@public.gmane.org \
    --cc=david.lanzendoerfer-Z7Kmv9EsliU@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).