On Wed, Oct 5, 2016 at 12:24 AM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > On Tue, Oct 04, 2016 at 11:46:21AM +0200, Mylène Josserand wrote: >> Add the documentation for dt-binding of the analog audiocodec >> driver for SUN8I SoC. >> >> Signed-off-by: Mylène Josserand <mylene.josserand@xxxxxxxxxxxxxxxxxx> >> --- >> .../devicetree/bindings/sound/sun8i-codec-analog.txt | 20 ++++++++++++++++++++ >> 1 file changed, 20 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/sound/sun8i-codec-analog.txt >> >> diff --git a/Documentation/devicetree/bindings/sound/sun8i-codec-analog.txt b/Documentation/devicetree/bindings/sound/sun8i-codec-analog.txt >> new file mode 100644 >> index 0000000..a03ec20 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/sound/sun8i-codec-analog.txt >> @@ -0,0 +1,20 @@ >> +* Allwinner A23/A33 Analog Codec >> + >> +This codec must be handled as a PRCM subnode. > > Like Mark was saying, you should probably reference the sun6i-prcm.txt > binding here > >> +Required properties: >> +- compatible: must be either "allwinner,sun8i-codec-analog" > > Our compatible prefix is <family>-<soc>, and using the older SoC that > introduced that block. > > In this case, that would be sun6i-a31, I think? sun6i-a31s actually, but a31s has extra line out controls, so the right one would be sun8i-a23. Both are listed in my original driver. ChenYu > > Thanks, > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
