On Thu, 2 May 2019 at 10:03, Mika Westerberg
<mika.westerberg@xxxxxxxxxxxxxxx> wrote:
>
> On Mon, Apr 29, 2019 at 03:12:08PM +0200, Ard Biesheuvel wrote:
> > @@ -160,13 +177,15 @@ static int mb86s70_gpio_probe(struct platform_device *pdev)
> > if (IS_ERR(gchip->base))
> > return PTR_ERR(gchip->base);
> >
> > - gchip->clk = devm_clk_get(&pdev->dev, NULL);
> > - if (IS_ERR(gchip->clk))
> > - return PTR_ERR(gchip->clk);
> > + if (!ACPI_COMPANION(&pdev->dev)) {
>
> Since you don't use the returned ACPI object, you can also use
> has_acpi_companion(&pdev->dev) here and other similar places.
>
> Regardless of that,
>
> Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
Thanks Mika. I will use has_acpi_companion() instead.
