[PATCH 2/2] ASoC: sunxi: compatibility for sun6i to SPDIF

Icenowy Zheng icenowy at aosc.xyz
Sat Jul 30 08:23:06 PDT 2016



30.07.2016, 23:20, "maxime.ripard at free-electrons.com" <maxime.ripard at free-electrons.com>:
> On Sat, Jul 30, 2016 at 10:52:45PM +0800, Icenowy Zheng wrote:
>>  > + if (of_device_is_compatible(pdev->dev.of_node,
>>  > + "allwinner,sun6i-a31-spdif")) {
>>  > + host->rst = devm_reset_control_get_optional(&pdev->dev, NULL);
>>  > + if (IS_ERR(host->rst) && PTR_ERR(host->rst) == -EPROBE_DEFER) {
>>  > + ret = -EPROBE_DEFER;
>>  > + dev_err(&pdev->dev, "Failed to get reset: %d\n", ret);
>>  > + goto err_disable_apb_clk;
>>  > + }
>>  > + if (!IS_ERR(host->rst))
>>  > + reset_control_deassert(host->rst);
>>  > + }
>>  > +
>>  I think you do not need the compatible.
>>  You can just detect whether the reset is present.
>
> That would weaken the error check. If we're running on the A31 and are
> missing our reset property, it would go unnoticed.
The reset property is in the SoC's dtsi file, so it won't be easily missing...

>
> Maxime
>
> --
> Maxime Ripard, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com



More information about the linux-arm-kernel mailing list