From: Jagan Teki <jagan@amarulasolutions.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/7] mmc: sunxi: Enable DM_MMC
Date: Sat, 19 Jan 2019 11:20:45 +0530 [thread overview]
Message-ID: <CAMty3ZC+uPb_awuCZ_cqEzb+PG9cn7Ym+ov=y4mYP1eRJxMGsg@mail.gmail.com> (raw)
In-Reply-To: <20190118174839.456f35d5@donnerap.cambridge.arm.com>
On Fri, Jan 18, 2019 at 11:18 PM Andre Przywara <andre.przywara@arm.com> wrote:
>
> On Fri, 18 Jan 2019 22:11:36 +0530
> Jagan Teki <jagan@amarulasolutions.com> wrote:
>
> Hi,
>
> > On Fri, Jan 18, 2019 at 6:00 PM Andre Przywara
> > <andre.przywara@arm.com> wrote:
> > >
> > > On Fri, 18 Jan 2019 07:17:41 -0500
> > > Tom Rini <trini@konsulko.com> wrote:
> > >
> > > > On Fri, Jan 18, 2019 at 11:53:49AM +0000, Andre Przywara wrote:
> > > > > On Thu, 17 Jan 2019 22:39:44 +0530
> > > > > Jagan Teki <jagan@amarulasolutions.com> wrote:
> > > > >
> > > > > > V2 for previous version[1] changes, for enabling DM_MMC
> > > > > > on Allwinner platform.
> > > > >
> > > > > So this is a neat and simple solution to the DM_MMC problem, to
> > > > > the point where I am wondering why we actually need all those
> > > > > DT driven clock and reset drivers in the first place.
> > > > > But as I understand using plat data in this way is somewhat
> > > > > frowned upon and considered deprecated (although it makes a lot
> > > > > of sense in this context).
> > > > >
> > > > > Also, isn't this series independent from the clock gates/reset
> > > > > patches? So why do you pile them on top of each other in
> > > > > sunxi/next?
> > > > >
> > > > > If we really want to have the full featured DT driven clock and
> > > > > reset support, why not use it together:
> > > > > We keep the current mod clock support in the MMC driver, but
> > > > > use the newly introduced clock gates and reset support via the
> > > > > new clock driver, mostly replacing this series. This would give
> > > > > us some test coverage of the new clock driver, while still
> > > > > avoiding to rush the MMC mod clock implementation.
> > > > >
> > > > > Does that make sense? Happy to bake some patches for that on the
> > > > > weekend.
> > > > >
> > > > > Btw: After talking to Tom on IRC, the DM_MMC deadline is
> > > > > actually _after_ the 2019.04 release, so we don't need to have
> > > > > DM_MMC support in this merge window.
> > > >
> > > > To be clearer, I plan to mark as BROKEN and start saying we're
> > > > going to remove stuff that isn't migrated, after the release. So
> > > > it would be good to get things moved this release that can be
> > > > moved this release. Trying to use sunxi w/o MMC isn't going to be
> > > > fun :)
> > >
> > > Understood. I just gave it a quick try and it is actually quite
> > > easy: We are pretty good already regarding gate clocks and resets,
> > > with the new DM_CLK driver (v2 on the list). And using them in
> > > sunxi_mmc.c is a piece of cake and very clean.
> > > We just need to keep the MMC mod clock hack in (which this series
> > > here does as well), but can still enable DM_MMC.
> > > And for the next merge window we can tackle this by implementing the
> > > MMC mod clock properly in the clock driver, then replacing the hack
> > > with the normal clk_get_by_name(); clk_set_rate(); sequence.
> >
> > I tried with ahb clock and reset, with v2 version of CLK series and
> > it's straightforward. but mmc clock may take some time along with
> > series of testing. So I just windup this, instead of making some noise
> > at last minute.
>
> What do you mean with that, exactly?
> Do you plan to take your platdata hack for 2019.04?
> I don't like the idea of hacking something up that has no future and
> will be reverted very soon anyway. Instead we should expose the
> foundation part of the clock driver to people now for testing (as you
> did by pushing it, thanks!), but including the MMC gates and resets.
> I have this code ready, just need to test it on some SoCs this evening.
>
> I think taking this change is the best compromise between changing
> not too many things at once, yet still exposing new code to the users
> for testing.
>
> And yes, the MMC mod clock is somewhat of a beast, but if we have
> just this in the next release, it should be easier to debug than when
> we expose the whole of the new clock framework to MMC by then only.
This is I was thinking too, in fact I have created next version wrt
CLK support which may override your recent changes. May be I can
prepare and by combining both. what do you say?
next prev parent reply other threads:[~2019-01-19 5:50 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-17 17:09 [U-Boot] [PATCH v2 0/7] mmc: sunxi: Enable DM_MMC Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 1/7] mmc: sunxi: Configure reset support for DM_MMC Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 2/7] mmc: sunxi: Add A83T emmc compatible Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 3/7] mmc: sunxi: Add mmc, emmc H5/A64 compatible Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 4/7] mmc: sunxi: Add DM_MMC support for H6 Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 5/7] mmc: sunxi: Add DM_MMC support for A80 Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 6/7] arm: sunxi: Enable DM_MMC Jagan Teki
2019-01-17 17:09 ` [U-Boot] [PATCH v2 7/7] arm: dts: sunxi: Enumerate MMC2 as MMC1 Jagan Teki
2019-01-18 11:53 ` [U-Boot] [PATCH v2 0/7] mmc: sunxi: Enable DM_MMC Andre Przywara
2019-01-18 12:17 ` Tom Rini
2019-01-18 12:30 ` Andre Przywara
2019-01-18 16:41 ` Jagan Teki
2019-01-18 17:48 ` Andre Przywara
2019-01-19 5:50 ` Jagan Teki [this message]
2019-01-19 10:17 ` André Przywara
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='CAMty3ZC+uPb_awuCZ_cqEzb+PG9cn7Ym+ov=y4mYP1eRJxMGsg@mail.gmail.com' \
--to=jagan@amarulasolutions.com \
--cc=u-boot@lists.denx.de \
/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 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.