linux-kernel - Re: [linux-sunxi] [PATCH v2] arm64: dts: allwinner: a64: Remove unused SPDIF sound card

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJiuCcdgDd=xrGU6iDjo=Og+OW=4cbrLG_NWZpzRZXdjJbnRXQ@mail.gmail.com>
Date:   Tue, 28 Apr 2020 16:50:54 +0200
From:   Clément Péron <peron.clem@...il.com>
To:     Samuel Holland <samuel@...lland.org>
Cc:     Maxime Ripard <mripard@...nel.org>, Chen-Yu Tsai <wens@...e.org>,
        Rob Herring <robh+dt@...nel.org>,
        devicetree <devicetree@...r.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        linux-sunxi <linux-sunxi@...glegroups.com>,
        Marcus Cooper <codekipper@...il.com>
Subject: Re: [linux-sunxi] [PATCH v2] arm64: dts: allwinner: a64: Remove
 unused SPDIF sound card

Hi,

On Sun, 26 Apr 2020 at 18:12, Samuel Holland <samuel@...lland.org> wrote:
>
> As of v5.7-rc2, Linux now prints the following message at boot:
>
>   [   33.848525] platform sound_spdif: deferred probe pending
>
> This is because sound_spdif is waiting on its CPU DAI &spdif to probe,
> but &spdif is disabled in the device tree.
>
> Exposure of the SPDIF pin is board-specific functionality, so the sound
> card and codec DAI belong in the individual board DTS, not the SoC DTSI.
> In fact, no in-tree A64 board DTS enables &spdif, so let's remove the
> card and DAI entirely.

CC: Marcus Cooper <codekipper@...il.com>

Without any value :
Acked-by: Clément Péron <peron.clem@...il.com>

Regards,
Clement

>
> This reverts commit 78e071370a86473f25923e03b51cbbadacf8be0f.
>
> Signed-off-by: Samuel Holland <samuel@...lland.org>
> ---
>
> Changes since v1:
>  - Remove the node instead of disabling it
>  - Also remove the codec DAI
>
> ---
>  arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 18 ------------------
>  1 file changed, 18 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
> index 31143fe64d91..c26cc1fcaffd 100644
> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
> @@ -154,24 +154,6 @@ link_codec: simple-audio-card,codec {
>                 };
>         };
>
> -       sound_spdif {
> -               compatible = "simple-audio-card";
> -               simple-audio-card,name = "On-board SPDIF";
> -
> -               simple-audio-card,cpu {
> -                       sound-dai = <&spdif>;
> -               };
> -
> -               simple-audio-card,codec {
> -                       sound-dai = <&spdif_out>;
> -               };
> -       };
> -
> -       spdif_out: spdif-out {
> -               #sound-dai-cells = <0>;
> -               compatible = "linux,spdif-dit";
> -       };
> -
>         timer {
>                 compatible = "arm,armv8-timer";
>                 allwinner,erratum-unknown1;
> --
> 2.24.1
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@...glegroups.com.
> To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/20200426161605.52121-1-samuel%40sholland.org.