Re: [PATCH v4 0/3] mmc: tmio_mmc: Add support for RZ/A2
On 24 October 2018 at 00:28, Chris Brandt wrote: > On Tuesday, October 23, 2018, Ulf Hansson wrote: >> On 12 October 2018 at 14:29, Chris Brandt >> wrote: >> > Basically the same HW block that was used in R-Car Gen 3 is used in >> > RZ/A2 (with only a couple small differences). >> > >> > Not sure if you're going to like the Kconfig change or not. >> > >> > Chris Brandt (3): >> > clk: renesas: r7s9210: Add SDHI clocks >> > mmc: renesas_sdhi_internal_dmac: Add R7S9210 support >> > dt-bindings: mmc: tmio_mmc: Document Renesas R7S9210 >> > >> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 3 ++- >> > drivers/clk/renesas/r7s9210-cpg-mssr.c | 5 >> > drivers/mmc/host/Kconfig | 5 ++-- >> > drivers/mmc/host/renesas_sdhi_internal_dmac.c | 28 >> -- >> > 4 files changed, 36 insertions(+), 5 deletions(-) >> > >> > -- >> > 2.16.1 >> >> This looks good to me. So unless someone objects I intend to queue >> this up for 4.21, once 4.20 rc1 is out. Or perhaps the clock patch >> should go via the clock tree? >> >> Thanks! >> Uffe > > FYI, > > I just tried to apply my patches to what is in -next. > It looks likes Fabrizio beat me to it (RZ/G1C has the same controller), > so there will be conflicts because we were both be modifying the same > lines in Kconfig and in the driver (adding the compatible strings). > > It also looks like Fabrizio started using the name > "renesas,sdhi-mmc-" (as in, added the '-mmc'), but I'll probably > just keep mine the way it is (no '-mmc'). > > > So if you don't feel like fixing it up, let me know and I'll send an update. Please send an update. Kind regards Uffe
Re: [PATCH v4 0/3] mmc: tmio_mmc: Add support for RZ/A2
Hi Ulf,
On Tue, Oct 23, 2018 at 5:02 PM Ulf Hansson wrote:
> On 12 October 2018 at 14:29, Chris Brandt wrote:
> > Basically the same HW block that was used in R-Car Gen 3 is used in
> > RZ/A2 (with only a couple small differences).
> >
> > Not sure if you're going to like the Kconfig change or not.
> >
> > Chris Brandt (3):
> > clk: renesas: r7s9210: Add SDHI clocks
> > mmc: renesas_sdhi_internal_dmac: Add R7S9210 support
> > dt-bindings: mmc: tmio_mmc: Document Renesas R7S9210
> >
> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 3 ++-
> > drivers/clk/renesas/r7s9210-cpg-mssr.c | 5
> > drivers/mmc/host/Kconfig | 5 ++--
> > drivers/mmc/host/renesas_sdhi_internal_dmac.c | 28
> > --
> > 4 files changed, 36 insertions(+), 5 deletions(-)
> >
> > --
> > 2.16.1
>
> This looks good to me. So unless someone objects I intend to queue
> this up for 4.21, once 4.20 rc1 is out. Or perhaps the clock patch
> should go via the clock tree?
Correct. Please do not take the clock patch.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
RE: [PATCH v4 0/3] mmc: tmio_mmc: Add support for RZ/A2
On Tuesday, October 23, 2018, Ulf Hansson wrote: > On 12 October 2018 at 14:29, Chris Brandt > wrote: > > Basically the same HW block that was used in R-Car Gen 3 is used in > > RZ/A2 (with only a couple small differences). > > > > Not sure if you're going to like the Kconfig change or not. > > > > Chris Brandt (3): > > clk: renesas: r7s9210: Add SDHI clocks > > mmc: renesas_sdhi_internal_dmac: Add R7S9210 support > > dt-bindings: mmc: tmio_mmc: Document Renesas R7S9210 > > > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 3 ++- > > drivers/clk/renesas/r7s9210-cpg-mssr.c | 5 > > drivers/mmc/host/Kconfig | 5 ++-- > > drivers/mmc/host/renesas_sdhi_internal_dmac.c | 28 > -- > > 4 files changed, 36 insertions(+), 5 deletions(-) > > > > -- > > 2.16.1 > > This looks good to me. So unless someone objects I intend to queue > this up for 4.21, once 4.20 rc1 is out. Or perhaps the clock patch > should go via the clock tree? > > Thanks! > Uffe FYI, I just tried to apply my patches to what is in -next. It looks likes Fabrizio beat me to it (RZ/G1C has the same controller), so there will be conflicts because we were both be modifying the same lines in Kconfig and in the driver (adding the compatible strings). It also looks like Fabrizio started using the name "renesas,sdhi-mmc-" (as in, added the '-mmc'), but I'll probably just keep mine the way it is (no '-mmc'). So if you don't feel like fixing it up, let me know and I'll send an update. Chris
Re: [PATCH v4 0/3] mmc: tmio_mmc: Add support for RZ/A2
On 12 October 2018 at 14:29, Chris Brandt wrote: > Basically the same HW block that was used in R-Car Gen 3 is used in > RZ/A2 (with only a couple small differences). > > Not sure if you're going to like the Kconfig change or not. > > Chris Brandt (3): > clk: renesas: r7s9210: Add SDHI clocks > mmc: renesas_sdhi_internal_dmac: Add R7S9210 support > dt-bindings: mmc: tmio_mmc: Document Renesas R7S9210 > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 3 ++- > drivers/clk/renesas/r7s9210-cpg-mssr.c | 5 > drivers/mmc/host/Kconfig | 5 ++-- > drivers/mmc/host/renesas_sdhi_internal_dmac.c | 28 > -- > 4 files changed, 36 insertions(+), 5 deletions(-) > > -- > 2.16.1 This looks good to me. So unless someone objects I intend to queue this up for 4.21, once 4.20 rc1 is out. Or perhaps the clock patch should go via the clock tree? Thanks! Uffe
[PATCH v4 0/3] mmc: tmio_mmc: Add support for RZ/A2
Basically the same HW block that was used in R-Car Gen 3 is used in RZ/A2 (with only a couple small differences). Not sure if you're going to like the Kconfig change or not. Chris Brandt (3): clk: renesas: r7s9210: Add SDHI clocks mmc: renesas_sdhi_internal_dmac: Add R7S9210 support dt-bindings: mmc: tmio_mmc: Document Renesas R7S9210 Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 3 ++- drivers/clk/renesas/r7s9210-cpg-mssr.c | 5 drivers/mmc/host/Kconfig | 5 ++-- drivers/mmc/host/renesas_sdhi_internal_dmac.c | 28 -- 4 files changed, 36 insertions(+), 5 deletions(-) -- 2.16.1
