[PATCH v2 01/46] panel/orisetech-otm8009a: Use refcounted allocation in place of devm_kzalloc()

[Anusha Srivatsa]

On Tue, Jun 3, 2025 at 2:11 AM Geert Uytterhoeven <geert at linux-m68k.org>
wrote:

> Hi Anusha,
>
> On Fri, 30 May 2025 at 20:28, Anusha Srivatsa <asrivats at redhat.com> wrote:
> > Move to using the new API devm_drm_panel_alloc() to allocate the
> > panel.
> >
> > Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>
> > Signed-off-by: Anusha Srivatsa <asrivats at redhat.com>
>
> Thanks for your patch!
>
> I have a generic comment that applies to all patches in the series.
>
> > --- a/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c
> > +++ b/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c
> > @@ -424,9 +424,11 @@ static int otm8009a_probe(struct mipi_dsi_device
> *dsi)
> >         struct otm8009a *ctx;
> >         int ret;
> >
> > -       ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
>
> The old code used sizeof(*ctx) instead of an explicit type, for
> increased type-safety.
>
> > -       if (!ctx)
> > -               return -ENOMEM;
> > +       ctx = devm_drm_panel_alloc(dev, struct otm8009a, panel,
>
> Can't you use __typeof(*ctx) instead of the explicit type here, too?
>
> Hi Geert,
Your suggestion does improve the patch for sure. The series along with its
first two parts is already merged
and this change will have to go as a separate series.

Thanks,
Anusha

> > +                                  &otm8009a_drm_funcs,
> > +                                  DRM_MODE_CONNECTOR_DSI);
> > +       if (IS_ERR(ctx))
> > +               return PTR_ERR(ctx);
> >
> >         ctx->reset_gpio = devm_gpiod_get_optional(dev, "reset",
> GPIOD_OUT_LOW);
> >         if (IS_ERR(ctx->reset_gpio)) {
>
> Gr{oetje,eeting}s,
>
>                         Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 --
> geert at linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker.
> But
> when I'm talking to journalists I just say "programmer" or something like
> that.
>                                 -- Linus Torvalds
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250603/aa8ebc4d/attachment-0001.htm>