On Sun, Jan 31, 2021 at 7:06 PM Matthias Brugger <matthias.bgg@xxxxxxxxx> wrote: > On 29/01/2021 10:49, Hsin-Hsiung Wang wrote: > > + mt6359codec: mt6359codec { > > + }; > > I understand that the dmic-mode and mic-type-X depends on the actual board on > which it is used. In that case I think we should add mt6359codec node in the dts > instead of dtsi file. I'd advise to set these properties as well as otherwise we > get a (slightly misleading) warning in the driver. I feel it is better to include the node in dtsi to represent the whole MT6359 PMIC. We could either: - Set default values of these properties in the dtsi to avoid the warning message. - Or https://patchwork.kernel.org/project/alsa-devel/patch/20210202033557.1621029-1-tzungbi@xxxxxxxxxx/