[Lima] [PATCH 1/2] drm/lima: Clean up IRQ warnings

Qiang Yu yuq825 at gmail.com
Wed Apr 22 01:29:24 UTC 2020


Looks good for me, patch 1&2 are:
Reviewed-by: Qiang Yu <yuq825 at gmail.com>

Regards,
Qiang

On Wed, Apr 22, 2020 at 6:51 AM Robin Murphy <robin.murphy at arm.com> wrote:
>
> Use the optional form of platform_get_irq() for blocks that legitimately
> may not be present, to avoid getting an annoying barrage of spurious
> warnings for non-existent PPs on configurations like Mali-450 MP2.
>
> Signed-off-by: Robin Murphy <robin.murphy at arm.com>
> ---
>  drivers/gpu/drm/lima/lima_device.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/lima/lima_device.c b/drivers/gpu/drm/lima/lima_device.c
> index 247f51fd40a2..c334d297796a 100644
> --- a/drivers/gpu/drm/lima/lima_device.c
> +++ b/drivers/gpu/drm/lima/lima_device.c
> @@ -171,8 +171,10 @@ static void lima_regulator_fini(struct lima_device *dev)
>
>  static int lima_init_ip(struct lima_device *dev, int index)
>  {
> +       struct platform_device *pdev = to_platform_device(dev->dev);
>         struct lima_ip_desc *desc = lima_ip_desc + index;
>         struct lima_ip *ip = dev->ip + index;
> +       const char *irq_name = desc->irq_name;
>         int offset = desc->offset[dev->id];
>         bool must = desc->must_have[dev->id];
>         int err;
> @@ -183,8 +185,9 @@ static int lima_init_ip(struct lima_device *dev, int index)
>         ip->dev = dev;
>         ip->id = index;
>         ip->iomem = dev->iomem + offset;
> -       if (desc->irq_name) {
> -               err = platform_get_irq_byname(dev->pdev, desc->irq_name);
> +       if (irq_name) {
> +               err = must ? platform_get_irq_byname(pdev, irq_name) :
> +                            platform_get_irq_byname_optional(pdev, irq_name);
>                 if (err < 0)
>                         goto out;
>                 ip->irq = err;
> --
> 2.23.0.dirty
>


More information about the lima mailing list