On Thu, 21 Mar 2024, Ville Syrjala <[email protected]> wrote:
> From: Ville Syrjälä <[email protected]>
>
> The VBT backlight entries include various fields for i2c
> controlled backlights. These have been obsoleted at some
> unknown point in time, but let's define them anyway so that
> we have a full picture of what has been in there.
>
> Signed-off-by: Ville Syrjälä <[email protected]>

Reviewed-by: Jani Nikula <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_vbt_defs.h | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h 
> b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> index 5da8ae407dfd..bcd1b239eb65 100644
> --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> @@ -881,11 +881,12 @@ struct bdb_lvds_lfp_data_tail {
>  struct lfp_backlight_data_entry {
>       u8 type:2;
>       u8 active_low_pwm:1;
> -     u8 obsolete1:5;
> +     u8 i2c_pin:3;                                           /* obsolete 
> since ? */
> +     u8 i2c_speed:2;                                         /* obsolete 
> since ? */
>       u16 pwm_freq_hz;
>       u8 min_brightness;                                      /* ???-233 */
> -     u8 obsolete2;
> -     u8 obsolete3;
> +     u8 i2c_address;                                         /* obsolete 
> since ? */
> +     u8 i2c_command;                                         /* obsolete 
> since ? */
>  } __packed;
>  
>  struct lfp_backlight_control_method {

-- 
Jani Nikula, Intel

Reply via email to