Commit 6cb9644d authored by Boris Brezillon's avatar Boris Brezillon Committed by Thierry Reding

backlight: pwm_bl: Use pwm_get_args() where appropriate

The PWM framework has clarified the concept of reference PWM config (the
platform dependent config retrieved from the DT or the PWM lookup table)
and real PWM state.

Use pwm_get_args() when the PWM user wants to retrieve this reference
config and not the current state.

This is part of the rework allowing the PWM framework to support
hardware readout and expose real PWM state even when the PWM has just
been requested (before the user calls pwm_config/enable/disable()).
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@free-electrons.com>
Signed-off-by: default avatarThierry Reding <thierry.reding@gmail.com>
parent 717c18f0
...@@ -201,6 +201,7 @@ static int pwm_backlight_probe(struct platform_device *pdev) ...@@ -201,6 +201,7 @@ static int pwm_backlight_probe(struct platform_device *pdev)
struct device_node *node = pdev->dev.of_node; struct device_node *node = pdev->dev.of_node;
struct pwm_bl_data *pb; struct pwm_bl_data *pb;
int initial_blank = FB_BLANK_UNBLANK; int initial_blank = FB_BLANK_UNBLANK;
struct pwm_args pargs;
int ret; int ret;
if (!data) { if (!data) {
...@@ -306,13 +307,20 @@ static int pwm_backlight_probe(struct platform_device *pdev) ...@@ -306,13 +307,20 @@ static int pwm_backlight_probe(struct platform_device *pdev)
dev_dbg(&pdev->dev, "got pwm for backlight\n"); dev_dbg(&pdev->dev, "got pwm for backlight\n");
/*
* FIXME: pwm_apply_args() should be removed when switching to
* the atomic PWM API.
*/
pwm_apply_args(pb->pwm);
/* /*
* The DT case will set the pwm_period_ns field to 0 and store the * The DT case will set the pwm_period_ns field to 0 and store the
* period, parsed from the DT, in the PWM device. For the non-DT case, * period, parsed from the DT, in the PWM device. For the non-DT case,
* set the period from platform data if it has not already been set * set the period from platform data if it has not already been set
* via the PWM lookup table. * via the PWM lookup table.
*/ */
pb->period = pwm_get_period(pb->pwm); pwm_get_args(pb->pwm, &pargs);
pb->period = pargs.period;
if (!pb->period && (data->pwm_period_ns > 0)) if (!pb->period && (data->pwm_period_ns > 0))
pb->period = data->pwm_period_ns; pb->period = data->pwm_period_ns;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment