Re: [PATCH v1 1/8] ARM: dts: imx6qdl-vicut1/vicutgo: Set default backlight brightness to maximum

2022-04-05 Thread Shawn Guo
On Mon, Feb 21, 2022 at 10:53:05AM +0100, Oleksij Rempel wrote:
> From: David Jander 
> 
> Recover default behavior of the device and set maximal brightness
> 
> Signed-off-by: David Jander 
> Signed-off-by: Oleksij Rempel 

Applied, thanks!


Re: [PATCH v1 1/8] ARM: dts: imx6qdl-vicut1/vicutgo: Set default backlight brightness to maximum

2022-04-05 Thread Shawn Guo
On Wed, Apr 6, 2022 at 9:23 AM Shawn Guo  wrote:
>
> On Mon, Feb 21, 2022 at 10:53:05AM +0100, Oleksij Rempel wrote:
> > From: David Jander 
> >
> > Recover default behavior of the device and set maximal brightness
> >
> > Signed-off-by: David Jander 
> > Signed-off-by: Oleksij Rempel 
> > ---
> >  arch/arm/boot/dts/imx6dl-victgo.dts   | 2 +-
> >  arch/arm/boot/dts/imx6qdl-vicut1.dtsi | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/imx6dl-victgo.dts 
> > b/arch/arm/boot/dts/imx6dl-victgo.dts
> > index 227c952543d4..e6134efbfabd 100644
> > --- a/arch/arm/boot/dts/imx6dl-victgo.dts
> > +++ b/arch/arm/boot/dts/imx6dl-victgo.dts
> > @@ -28,7 +28,7 @@ backlight: backlight {
> >   pwms = <&pwm1 0 500 0>;
> >   brightness-levels = <0 16 64 255>;
> >   num-interpolated-steps = <16>;
> > - default-brightness-level = <1>;
> > + default-brightness-level = <48>;
>
> Please take a look at the documentation of the property.
>
>   default-brightness-level:
> description:
>   The default brightness level (index into the array defined by the
>   "brightness-levels" property).

Ah, never mind, I missed 'num-interpolated-steps' there.

Shawn


Re: [PATCH v1 1/8] ARM: dts: imx6qdl-vicut1/vicutgo: Set default backlight brightness to maximum

2022-04-05 Thread Shawn Guo
On Mon, Feb 21, 2022 at 10:53:05AM +0100, Oleksij Rempel wrote:
> From: David Jander 
> 
> Recover default behavior of the device and set maximal brightness
> 
> Signed-off-by: David Jander 
> Signed-off-by: Oleksij Rempel 
> ---
>  arch/arm/boot/dts/imx6dl-victgo.dts   | 2 +-
>  arch/arm/boot/dts/imx6qdl-vicut1.dtsi | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6dl-victgo.dts 
> b/arch/arm/boot/dts/imx6dl-victgo.dts
> index 227c952543d4..e6134efbfabd 100644
> --- a/arch/arm/boot/dts/imx6dl-victgo.dts
> +++ b/arch/arm/boot/dts/imx6dl-victgo.dts
> @@ -28,7 +28,7 @@ backlight: backlight {
>   pwms = <&pwm1 0 500 0>;
>   brightness-levels = <0 16 64 255>;
>   num-interpolated-steps = <16>;
> - default-brightness-level = <1>;
> + default-brightness-level = <48>;

Please take a look at the documentation of the property.

  default-brightness-level:
description:
  The default brightness level (index into the array defined by the
  "brightness-levels" property).

Shawn

>   power-supply = <®_3v3>;
>   enable-gpios = <&gpio4 28 GPIO_ACTIVE_HIGH>;
>   };
> diff --git a/arch/arm/boot/dts/imx6qdl-vicut1.dtsi 
> b/arch/arm/boot/dts/imx6qdl-vicut1.dtsi
> index 1ac7e13249d2..c1d06bc28c67 100644
> --- a/arch/arm/boot/dts/imx6qdl-vicut1.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-vicut1.dtsi
> @@ -23,7 +23,7 @@ backlight: backlight {
>   pwms = <&pwm1 0 500 0>;
>   brightness-levels = <0 16 64 255>;
>   num-interpolated-steps = <16>;
> - default-brightness-level = <1>;
> + default-brightness-level = <48>;
>   power-supply = <®_3v3>;
>   enable-gpios = <&gpio4 28 GPIO_ACTIVE_HIGH>;
>   };
> -- 
> 2.30.2
>