[U-Boot] [PATCH v3 3/4] mmc: zynq: Determine base clock frequency via clock framework

[Stefan.Herbrechtsmeier at weidmueller.com]

> -----Ursprüngliche Nachricht-----
> Von: Michal Simek [mailto:michal.simek at xilinx.com]
> Gesendet: Dienstag, 10. Januar 2017 16:39
> 
> On 10.1.2017 15:05, Stefan.Herbrechtsmeier at weidmueller.com wrote:
> > Hi Michal,
> >
> >> -----Ursprüngliche Nachricht-----
> >> Von: Michal Simek [mailto:michal.simek at xilinx.com]
> >> Gesendet: Dienstag, 10. Januar 2017 14:30
> >> An: Herbrechtsmeier, Stefan; michal.simek at xilinx.com;
> >> jh80.chung at samsung.com; u-boot at lists.denx.de
> >> Cc: sjg at chromium.org; monstr at monstr.eu; swarren at nvidia.com
> >> Betreff: Re: AW: [PATCH v3 3/4] mmc: zynq: Determine base clock
> >> frequency via clock framework
> >>
> >> On 10.1.2017 10:16, Stefan.Herbrechtsmeier at weidmueller.com wrote:
> >>> Hi Michael
> >>>
> >>>> -----Ursprüngliche Nachricht-----
> >>>> Von: Michal Simek [mailto:michal.simek at xilinx.com]
> >>>> Gesendet: Dienstag, 6. Dezember 2016 13:59
> >>>> An: Jaehoon Chung; Herbrechtsmeier, Stefan; u-boot at lists.denx.de
> >>>> Cc: Michal Simek; Simon Glass; Albert Aribaud; Michal Simek;
> >>>> Stephen Warren
> >>>> Betreff: Re: [PATCH v3 3/4] mmc: zynq: Determine base clock
> >> frequency
> >>>> via clock framework
> >>>>
> >>>> On 5.12.2016 23:22, Jaehoon Chung wrote:
> >>>>> On 12/02/2016 10:24 PM, stefan.herbrechtsmeier at weidmueller.de
> >> wrote:
> >>>>>> From: Stefan Herbrechtsmeier
> >>>>>> <stefan.herbrechtsmeier at weidmueller.de>
> >>>>>>
> >>>>>> The zynq_sdhci controller driver use CONFIG_ZYNQ_SDHCI_MAX_FREQ
> >>>>>> as base clock frequency but this clock is not fixed and depends
> >>>>>> on
> >> the
> >>>>>> hardware configuration. Additionally the value of
> >>>>>> CONFIG_ZYNQ_SDHCI_MAX_FREQ doesn't match the real base clock
> >>>>>> frequency of SDIO_FREQ. Use the clock framework to determine the
> >>>> frequency at run time.
> >>>>>>
> >>>>>> Signed-off-by: Stefan Herbrechtsmeier
> >>>>>> <stefan.herbrechtsmeier at weidmueller.de>
> >>>>>
> >>>>>
> >>>>> Even if this patch is delegated to Michal, i picked to this.
> >>>>> Applied on u-boot-mmc.
> >>>>>
> >>>>
> >>>> Based on my reply in v2 this is not ideal patch
> >>>> http://lists.denx.de/pipermail/u-boot/2016-November/274310.html
> >>>>
> >>>> Not a problem to take these patches should mmc tree but please
> wait
> >>>> till you have Ack from me.
> >>>>
> >>> Will you ack the patch if I remove the zynq specific code and
> remove
> >> the CONFIG_CLK check with my patch series to move zynq clk driver to
> >> DM?
> >>
> >> Just going to look at our clk changes for zynq and it can be the
> part
> >> of this series.
> > Okay.
> >
> >> Do you have branch somewhere with pending stuff?
> >
> > I have no public branch.
> 
> ok. Can you please send it as the part of clk series you made?
> I see that at least this patch will be different.

Okay.



Kommanditgesellschaft - Sitz: Detmold - Amtsgericht Lemgo HRA 2790 - 
Komplementärin: Weidmüller Interface Führungsgesellschaft mbH - 
Sitz: Detmold - Amtsgericht Lemgo HRB 3924; 
Geschäftsführer: José Carlos Álvarez Tobar, Elke Eckstein, Dr. Peter Köhler, Jörg Timmermann;
USt-ID-Nr. DE124599660