Hi Nicolas, On Tue, 2019-05-07 at 14:24 +0900, Nicolas Boichat wrote: > On Fri, May 3, 2019 at 6:34 PM Hsin-Hsiung Wang > <hsin-hsiung.wang@xxxxxxxxxxxx> wrote: > > ... > > This adds compatible for the MediaTek MT6358 PMIC. > > > > Acked-for-MFD-by: Lee Jones <lee.jones@xxxxxxxxxx> > > Signed-off-by: Hsin-Hsiung Wang <hsin-hsiung.wang@xxxxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/mfd/mt6397.txt | 11 ++++++++--- > > 1 file changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/mfd/mt6397.txt b/Documentation/devicetree/bindings/mfd/mt6397.txt > > index 0ebd08af777d..62f1c17c7738 100644 > > --- a/Documentation/devicetree/bindings/mfd/mt6397.txt > > +++ b/Documentation/devicetree/bindings/mfd/mt6397.txt > > @@ -17,22 +17,27 @@ Documentation/devicetree/bindings/soc/mediatek/pwrap.txt > > This document describes the binding for MFD device and its sub module. > > > > Required properties: > > -compatible: "mediatek,mt6397" or "mediatek,mt6323" > > +compatible: > > + "mediatek,mt6323" for PMIC MT6323 > > + "mediatek,mt6358" for PMIC MT6358 > > + "mediatek,mt6397" for PMIC MT6397 > > > > Optional subnodes: > > > > - rtc > > Required properties: > > - - compatible: "mediatek,mt6397-rtc" > > + - compatible: "mediatek,mt6397-rtc" or "mediatek,mt6358-rtc" > > - regulators > > Required properties: > > - compatible: "mediatek,mt6397-regulator" > > see Documentation/devicetree/bindings/regulator/mt6397-regulator.txt > > + - compatible: "mediatek,mt6358-regulator" > > + see Documentation/devicetree/bindings/regulator/mt6358-regulator.txt > > - compatible: "mediatek,mt6323-regulator" > > see Documentation/devicetree/bindings/regulator/mt6323-regulator.txt > > - codec > > Required properties: > > - - compatible: "mediatek,mt6397-codec" > > + - compatible: "mediatek,mt6397-codec" or "mediatek,mt6358-sound" > > Sean had a question about this > (https://patchwork.kernel.org/patch/10846669/#22524299): > "why didn't we use a more consistent naming as mt6358-codec?" > The compatible name, "mediatek,mt6358-sound", is already in mt6358 audio driver which is already in the upstream codebase. After discussing with audio owner, we will submit another patch to fix it. Thanks. > > - clk > > Required properties: > > - compatible: "mediatek,mt6397-clk" > > -- > > 2.18.0 > > > > > > _______________________________________________ > > Linux-mediatek mailing list > > Linux-mediatek@xxxxxxxxxxxxxxxxxxx > > http://lists.infradead.org/mailman/listinfo/linux-mediatek
