Hi,
On Sun, Aug 02, 2020 at 04:32:07PM +0200, Sam Ravnborg wrote:
> On Sun, Aug 02, 2020 at 04:26:05PM +0200, Sebastian Reichel wrote:
> > On Sun, Aug 02, 2020 at 01:06:35PM +0200, Sam Ravnborg wrote:
> > > - Introduce backlight_{enable/disable)
> > > - Use get/set methods for backlight_properties
> > > - Drop redundant get_brightness() implementation
> > > The default implementation return the current brightness value
> > > - Use macro for backlight initialization
> > >
> > > Signed-off-by: Sam Ravnborg <s...@ravnborg.org>
> > > Cc: Tomi Valkeinen <tomi.valkei...@ti.com>
> > > Cc: Sebastian Reichel <sebastian.reic...@collabora.com>
> > > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> > > Cc: Zheng Bin <zhengbi...@huawei.com>
> > > Cc: Sam Ravnborg <s...@ravnborg.org>
> > > ---
> > > .../gpu/drm/omapdrm/displays/panel-dsi-cm.c | 35 ++++---------------
> > > 1 file changed, 6 insertions(+), 29 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
> > > b/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
> > > index 3484b5d4a91c..433e240896b3 100644
> > > --- a/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
> > > +++ b/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
> > > @@ -110,15 +110,10 @@ static void dsicm_bl_power(struct panel_drv_data
> > > *ddata, bool enable)
> > > else
> > > return;
> > >
> > > - if (enable) {
> > > - backlight->props.fb_blank = FB_BLANK_UNBLANK;
> > > - backlight->props.state = ~(BL_CORE_FBBLANK | BL_CORE_SUSPENDED);
> > > - backlight->props.power = FB_BLANK_UNBLANK;
> > > - } else {
> > > - backlight->props.fb_blank = FB_BLANK_NORMAL;
> > > - backlight->props.power = FB_BLANK_POWERDOWN;
> > > - backlight->props.state |= BL_CORE_FBBLANK | BL_CORE_SUSPENDED;
> > > - }
> > > + if (enable)
> > > + backlight_enable(backlight);
> > > + else
> > > + backlight_disable(backlight);
> > >
> > > backlight_update_status(backlight);
> >
> > backlight_update_status() is already called by backlight_enable/disable.
>
> Right, thanks.
> Dropped in v2.
>
> Let me know if you already have a similar patch and if I shall
> drop this.I did not touch the backlight bits and I can easily rebase my patches. I think this should be kept. > It would be nice to have the panel parts of omapdrm migrated in > this cycle. I recall you have 50+ patches pending. I plan to send only the first part and go step by step. -- Sebastian
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
