On 29/04/2019 16:29, Lucas Stach wrote:
This way the backlight can be referenced through its device node and enabling/disabling can be managed through the panel driver.
Is it possible to implement something similar to pwm_backlight_initial_power_state() to handle this?
backlight drivers already suffer from too much diversity so I prefer things like this to align behaviour with the (fairly heavilyly used) PWM driver if possible.
Daniel.
Signed-off-by: Lucas Stach <l.st...@pengutronix.de> --- drivers/video/backlight/rave-sp-backlight.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/video/backlight/rave-sp-backlight.c b/drivers/video/backlight/rave-sp-backlight.c index 462f14a1b19d..d296bfcf4396 100644 --- a/drivers/video/backlight/rave-sp-backlight.c +++ b/drivers/video/backlight/rave-sp-backlight.c @@ -48,15 +48,13 @@ static int rave_sp_backlight_probe(struct platform_device *pdev) struct device *dev = &pdev->dev; struct backlight_device *bd;- bd = devm_backlight_device_register(dev, pdev->name, dev->parent,+ bd = devm_backlight_device_register(dev, pdev->name, dev, dev_get_drvdata(dev->parent), &rave_sp_backlight_ops, &rave_sp_backlight_props); if (IS_ERR(bd)) return PTR_ERR(bd);- backlight_update_status(bd);- return 0; }
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
