[Lima] [PATCH 1/2] drm/lima: Clean up IRQ warnings
Qiang Yu
yuq825 at gmail.com
Fri Apr 24 13:06:38 UTC 2020
Applied to drm-misc-next.
Thanks,
Qiang
On Wed, Apr 22, 2020 at 9:29 AM Qiang Yu <yuq825 at gmail.com> wrote:
>
> 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