On Tue, Apr 24, 2018 at 11:53:15AM +0200, Geert Uytterhoeven wrote: > Hi Simon, > > On Tue, Apr 24, 2018 at 11:36 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > > On Mon, Apr 23, 2018 at 10:01:51AM +0200, Geert Uytterhoeven wrote: > >> On Mon, Apr 23, 2018 at 3:39 AM, Kuninori Morimoto > >> <kuninori.morimoto.gx@xxxxxxxxxxx> wrote: > >> > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > >> > > >> > CONFIG_SND_AUDIO_GRAPH_CARD is needed to use HDMI sound with video > >> > > >> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > >> > Tested-by: Nguyen Viet Dung <nv-dung@xxxxxxxxxxx> > >> > >> Thanks for your patch! > >> > >> > --- a/arch/arm64/configs/defconfig > >> > +++ b/arch/arm64/configs/defconfig > >> > @@ -440,6 +440,7 @@ CONFIG_SND_SOC_SAMSUNG=y > >> > CONFIG_SND_SOC_RCAR=m > >> > CONFIG_SND_SOC_AK4613=m > >> > CONFIG_SND_SIMPLE_CARD=y > >> > +CONFIG_SND_AUDIO_GRAPH_CARD=y > >> > >> Perhaps "m"? > > > > Do you suggest the same for renesas_defconfig? > > That is, patch 2/10? > > No. For {shmobile,renesas}_defconfig we make everything builtin. > For {multi_v7,arm64}_defconfig, we usually make non-critical devices modular. Thanks, that makes sense to me.