lkml.org 
[lkml]   [2018]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 8/8] ASoC: samsung,tm2-audio DT binding documentation update
    From
    Date
    On 02/06/2018 02:05 PM, Krzysztof Kozlowski wrote:
    > On Mon, Feb 5, 2018 at 4:44 PM, Sylwester Nawrocki
    > <s.nawrocki@samsung.com> wrote:
    >> This patch documents additional entries of the audio-codec and
    >> i2s-controller properties required for the HDMI audio support.
    >>
    >> Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
    >> ---
    >> .../devicetree/bindings/sound/samsung,tm2-audio.txt | 14 +++++++++-----
    >> 1 file changed, 9 insertions(+), 5 deletions(-)
    >>
    >> diff --git a/Documentation/devicetree/bindings/sound/samsung,tm2-audio.txt b/Documentation/devicetree/bindings/sound/samsung,tm2-audio.txt
    >> index 94442e5673b3..f5ccc12ddc00 100644
    >> --- a/Documentation/devicetree/bindings/sound/samsung,tm2-audio.txt
    >> +++ b/Documentation/devicetree/bindings/sound/samsung,tm2-audio.txt
    >> @@ -4,9 +4,13 @@ Required properties:
    >>
    >> - compatible : "samsung,tm2-audio"
    >> - model : the user-visible name of this sound complex
    >> - - audio-codec : the phandle of the wm5110 audio codec node,
    >> - as described in ../mfd/arizona.txt
    >> - - i2s-controller : the phandle of the I2S controller
    >> + - audio-codec : the first entry should be phandle of the wm5110 audio
    >> + codec node, as described in ../mfd/arizona.txt;
    >> + the second entry should be phandle of the HDMI
    >
    > AFAIU, this property is still optional so the wording should be
    > different. Maybe just add "optional properties" paragraph?

    I'd like to keep the second entries required in the documentation,
    and the driver would still be handling the case with just single
    entries.

    --
    Regards,
    Sylwester

    \
     
     \ /
      Last update: 2018-02-07 17:32    [W:2.357 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site