RE: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-25 Thread Flavio Suligoi
Hi Conor, ... > > > > > > The two properties: > > > > > > > > > > > > - max-brightness > > > > > > - default brightness > > > > > > > > > > > > are not really required, so they can be removed from the "required" > > > > > > section. > > > > > > > > > > Why are they not required? You need to provi

Re: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-24 Thread Conor Dooley
On Tue, Oct 24, 2023 at 07:53:38AM +, Flavio Suligoi wrote: > > On Mon, Oct 23, 2023 at 09:28:03AM +, Flavio Suligoi wrote: > > > > On Fri, Oct 20, 2023 at 03:54:33PM +0200, Flavio Suligoi wrote: > > > > > The two properties: > > > > > > > > > > - max-brightness > > > > > - default brightne

RE: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-24 Thread Flavio Suligoi
Hi Conor, ... > On Mon, Oct 23, 2023 at 09:28:03AM +, Flavio Suligoi wrote: > > > On Fri, Oct 20, 2023 at 03:54:33PM +0200, Flavio Suligoi wrote: > > > > The two properties: > > > > > > > > - max-brightness > > > > - default brightness > > > > > > > > are not really required, so they can be r

Re: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-23 Thread Conor Dooley
On Mon, Oct 23, 2023 at 09:28:03AM +, Flavio Suligoi wrote: > > On Fri, Oct 20, 2023 at 03:54:33PM +0200, Flavio Suligoi wrote: > > > The two properties: > > > > > > - max-brightness > > > - default brightness > > > > > > are not really required, so they can be removed from the "required" > > >

RE: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-23 Thread Flavio Suligoi
Hi Conor, ... > On Fri, Oct 20, 2023 at 03:54:33PM +0200, Flavio Suligoi wrote: > > The two properties: > > > > - max-brightness > > - default brightness > > > > are not really required, so they can be removed from the "required" > > section. > > Why are they not required? You need to provide an

Re: [PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-20 Thread Conor Dooley
Yo, On Fri, Oct 20, 2023 at 03:54:33PM +0200, Flavio Suligoi wrote: > The two properties: > > - max-brightness > - default brightness > > are not really required, so they can be removed from the "required" > section. Why are they not required? You need to provide an explanation. > Other change

[PATCH 1/1] dt-bindings: backlight: mp3309c: remove two required properties

2023-10-20 Thread Flavio Suligoi
The two properties: - max-brightness - default brightness are not really required, so they can be removed from the "required" section. Other changes: - improve the backlight working mode description, in the "description" section - update the example, removing the "max-brightness" and introduc