On Tue, Apr 1, 2025 at 3:22 PM Anand Moon <linux.am...@gmail.com> wrote:
>
> Hi Kever,
>
> On Tue, 1 Apr 2025 at 09:11, Anand Moon <linux.am...@gmail.com> wrote:
> >
> > Hi Kever,
> >
> > On Tue, 1 Apr 2025 at 08:30, Kever Yang <kever.y...@rock-chips.com> wrote:
> > >
> > > Hi Anand,
> > >
> > > On 2025/3/30 21:33, Anand Moon wrote:
> > > > Hi Chen-Yu
> > > >
> > > > On Wed, 5 Feb 2025 at 07:39, Chen-Yu Tsai <w...@kernel.org> wrote:
> > > >> From: Chen-Yu Tsai <w...@csie.org>
> > > >>
> > > >> The IO domain driver controls the I/O voltage for various pins,
> > > >> MMC included.
> > > >>
> > > >> Enable it by default for all supported Rockchip SoCs.
> > > >>
> > > >> Signed-off-by: Chen-Yu Tsai <w...@csie.org>
> > > > Reviewed-by: Anand Moon <linux.am...@gmail.com>
> > > >
> > > > Can we enable following config default for all the boards
> > > >
> > > > CONFIG_MMC_DW=y
> > > > CONFIG_MMC_DW_ROCKCHIP=y
> > > > CONFIG_MMC_SDHCI=y
> > > These two driver, SD card and eMMC, are board specific, not all the
> > > board has enable all these two interface.
> > >
> > Ok,I thought it should reduce the configuration options on all the boards.
> But can we implement this for each SoC, such as the RK3399 and RK3588?"
Of all the defconfigs that have ARCH_ROCKCHIP, only three don't have MMC_DW:
configs/coolpi-cm5-genbook-rk3588_defconfig
configs/evb-rv1108_defconfig
configs/geekbox_defconfig
So maybe we could have the default by y, and negate the option where
it makes sense.
For MMC_SDHCI there would be more defconfigs, but only a couple SoCs
support this controller, and we could narrow it down to those.
ChenYu
