fsl_ssi: fix of_property_read_u32_array return value check
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Wed, 4 Mar 2015 21:48:30 +0000 (22:48 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 5 Mar 2015 01:01:12 +0000 (01:01 +0000)
of_property_read_u32_array returns 0 on success,
so the return value shouldn't be inverted twice,
first on assignment then in condition expression.

Signed-off-by: Maciej Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_ssi.c

index d7365c5d7ec0b42dc276b45e2296b55725484df1..134388f7d1b853f6bfcccc49d858676b32ee476b 100644 (file)
@@ -1227,7 +1227,7 @@ static int fsl_ssi_imx_probe(struct platform_device *pdev,
        ssi_private->dma_params_tx.addr = ssi_private->ssi_phys + CCSR_SSI_STX0;
        ssi_private->dma_params_rx.addr = ssi_private->ssi_phys + CCSR_SSI_SRX0;
 
-       ret = !of_property_read_u32_array(np, "dmas", dmas, 4);
+       ret = of_property_read_u32_array(np, "dmas", dmas, 4);
        if (ssi_private->use_dma && !ret && dmas[2] == IMX_DMATYPE_SSI_DUAL) {
                ssi_private->use_dual_fifo = true;
                /* When using dual fifo mode, we need to keep watermark