Hi Michal, > -----Ursprüngliche Nachricht----- > Von: Michal Simek [mailto:michal.si...@xilinx.com] > Gesendet: Dienstag, 10. Januar 2017 14:30 > An: Herbrechtsmeier, Stefan; michal.si...@xilinx.com; > jh80.ch...@samsung.com; u-boot@lists.denx.de > Cc: s...@chromium.org; mon...@monstr.eu; swar...@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.herbrechtsme...@weidmueller.com wrote: > > Hi Michael > > > >> -----Ursprüngliche Nachricht----- > >> Von: Michal Simek [mailto:michal.si...@xilinx.com] > >> Gesendet: Dienstag, 6. Dezember 2016 13:59 > >> An: Jaehoon Chung; Herbrechtsmeier, Stefan; u-boot@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.herbrechtsme...@weidmueller.de > wrote: > >>>> From: Stefan Herbrechtsmeier > >>>> <stefan.herbrechtsme...@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.herbrechtsme...@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. Regards, Stefan 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
