On Wed, 10 Nov 2021 at 20:16, Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > > V3M handles SDnH differently than other Gen3 SoCs, so let's add a > separate entry for that. This will allow better SDnH handling in the > future. > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> Kind regards Uffe > --- > Changes since RFC v1: > * new patch > > drivers/mmc/host/renesas_sdhi.h | 3 +++ > drivers/mmc/host/renesas_sdhi_internal_dmac.c | 21 +++++++++++++++++++ > 2 files changed, 24 insertions(+) > > diff --git a/drivers/mmc/host/renesas_sdhi.h b/drivers/mmc/host/renesas_sdhi.h > index 0c45e82ff0de..cd82420677cc 100644 > --- a/drivers/mmc/host/renesas_sdhi.h > +++ b/drivers/mmc/host/renesas_sdhi.h > @@ -18,6 +18,8 @@ struct renesas_sdhi_scc { > u32 tap_hs400_4tap; /* sampling clock position for HS400 (4 TAP) */ > }; > > +#define SDHI_FLAG_NEED_CLKH_FALLBACK BIT(0) > + > struct renesas_sdhi_of_data { > unsigned long tmio_flags; > u32 tmio_ocr_mask; > @@ -31,6 +33,7 @@ struct renesas_sdhi_of_data { > int taps_num; > unsigned int max_blk_count; > unsigned short max_segs; > + unsigned long sdhi_flags; > }; > > #define SDHI_CALIB_TABLE_MAX 32 > diff --git a/drivers/mmc/host/renesas_sdhi_internal_dmac.c b/drivers/mmc/host/renesas_sdhi_internal_dmac.c > index 7660f7ea74dd..9d2c600fd4ce 100644 > --- a/drivers/mmc/host/renesas_sdhi_internal_dmac.c > +++ b/drivers/mmc/host/renesas_sdhi_internal_dmac.c > @@ -125,6 +125,22 @@ static const struct renesas_sdhi_of_data of_data_rcar_gen3 = { > /* DMAC can handle 32bit blk count but only 1 segment */ > .max_blk_count = UINT_MAX / TMIO_MAX_BLK_SIZE, > .max_segs = 1, > + .sdhi_flags = SDHI_FLAG_NEED_CLKH_FALLBACK, > +}; > + > +static const struct renesas_sdhi_of_data of_data_rcar_gen3_no_fallback = { > + .tmio_flags = TMIO_MMC_HAS_IDLE_WAIT | TMIO_MMC_CLK_ACTUAL | > + TMIO_MMC_HAVE_CBSY | TMIO_MMC_MIN_RCAR2, > + .capabilities = MMC_CAP_SD_HIGHSPEED | MMC_CAP_SDIO_IRQ | > + MMC_CAP_CMD23 | MMC_CAP_WAIT_WHILE_BUSY, > + .capabilities2 = MMC_CAP2_NO_WRITE_PROTECT | MMC_CAP2_MERGE_CAPABLE, > + .bus_shift = 2, > + .scc_offset = 0x1000, > + .taps = rcar_gen3_scc_taps, > + .taps_num = ARRAY_SIZE(rcar_gen3_scc_taps), > + /* DMAC can handle 32bit blk count but only 1 segment */ > + .max_blk_count = UINT_MAX / TMIO_MAX_BLK_SIZE, > + .max_segs = 1, > }; > > static const u8 r8a7796_es13_calib_table[2][SDHI_CALIB_TABLE_MAX] = { > @@ -214,6 +230,10 @@ static const struct renesas_sdhi_of_data_with_quirks of_r8a77965_compatible = { > .quirks = &sdhi_quirks_r8a77965, > }; > > +static const struct renesas_sdhi_of_data_with_quirks of_r8a77970_compatible = { > + .of_data = &of_data_rcar_gen3_no_fallback, > +}; > + > static const struct renesas_sdhi_of_data_with_quirks of_r8a77980_compatible = { > .of_data = &of_data_rcar_gen3, > .quirks = &sdhi_quirks_nohs400, > @@ -235,6 +255,7 @@ static const struct of_device_id renesas_sdhi_internal_dmac_of_match[] = { > { .compatible = "renesas,sdhi-r8a7796", .data = &of_rcar_gen3_compatible, }, > { .compatible = "renesas,sdhi-r8a77961", .data = &of_r8a77961_compatible, }, > { .compatible = "renesas,sdhi-r8a77965", .data = &of_r8a77965_compatible, }, > + { .compatible = "renesas,sdhi-r8a77970", .data = &of_r8a77970_compatible, }, > { .compatible = "renesas,sdhi-r8a77980", .data = &of_r8a77980_compatible, }, > { .compatible = "renesas,sdhi-r8a77990", .data = &of_r8a77990_compatible, }, > { .compatible = "renesas,rcar-gen3-sdhi", .data = &of_rcar_gen3_compatible, }, > -- > 2.30.2 >