[PATCH 3/5] gpu: drm: adp: Add a backlight driver for the Summit LCD
Sasha Finkelstein
fnkl.kernel at gmail.com
Mon Nov 25 11:28:18 UTC 2024
On Mon, 25 Nov 2024 at 09:45, Neil Armstrong <neil.armstrong at linaro.org> wrote:
> > +static int summit_suspend(struct device *dev)
> > +{
> > + struct summit_data *panel = dev_get_drvdata(dev);
> > +
> > + int err = mipi_dsi_dcs_set_display_brightness(panel->dsi, 0);
> > +
> > + if (err < 0)
> > + return err;
> > + return 0;
>
> Just return err here, add a common function to set a brighness value and
> avoid duplicate code like here.
I felt that mipi_dsi_dcs_set_display_brightness is common enough, is it not?
Ack on all other changes, will be done for v2.
More information about the dri-devel
mailing list