Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-20 Thread Rob Herring
On Fri, 14 Oct 2022 15:51:04 -0500, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > A meta-schema check for this is pending, so hopefully the last time to > fix the

Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-17 Thread Thierry Reding
On Fri, Oct 14, 2022 at 03:51:04PM -0500, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > A meta-schema check for this is pending, so hopefully the last time to > f

Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-17 Thread Nicolas Ferre
On 14/10/2022 at 22:51, Rob Herring wrote: There's no reason to have "status" properties in examples. "okay" is the default, and "disabled" turns off some schema checks ('required' specifically). A meta-schema check for this is pending, so hopefully the last time to fix these. Fix the indentati

Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-16 Thread Vinod Koul
On 14-10-22, 15:51, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > A meta-schema check for this is pending, so hopefully the last time to > fix these. > > Fix the

Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-15 Thread Jonathan Cameron
On Fri, 14 Oct 2022 15:51:04 -0500 Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > A meta-schema check for this is pending, so hopefully the last time to > fix th

Re: [PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-15 Thread Krzysztof Kozlowski
On 14/10/2022 16:51, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > A meta-schema check for this is pending, so hopefully the last time to > fix these. > > Fix th

[PATCH] dt-bindings: Remove "status" from schema examples, again

2022-10-14 Thread Rob Herring
There's no reason to have "status" properties in examples. "okay" is the default, and "disabled" turns off some schema checks ('required' specifically). A meta-schema check for this is pending, so hopefully the last time to fix these. Fix the indentation in intel,phy-thunderbay-emmc while we're h

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-23 Thread Rob Herring
On Tue, 20 Jul 2021 11:20:25 -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Document

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-21 Thread Linus Walleij
On Tue, Jul 20, 2021 at 7:20 PM Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). All of mine are mistakes anyway. Reviewed-by: Linus Walleij Thanks for fixing this u

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-21 Thread Rob Herring
On Wed, Jul 21, 2021 at 2:33 AM Philipp Zabel wrote: > > Hi Rob, > > On Tue, 2021-07-20 at 11:20 -0600, Rob Herring wrote: > > There's no reason to have "status" properties in examples. "okay" is the > > default, and "disabled" turns off some schema checks ('required' > > specifically). > > Is thi

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-21 Thread Oleksij Rempel
On Tue, Jul 20, 2021 at 11:20:25AM -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Do

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-21 Thread Philipp Zabel
Hi Rob, On Tue, 2021-07-20 at 11:20 -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). Is this documented somewhere? If not, should it be? (Maybe in writing- schem

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-21 Thread Rui Miguel Silva
On Tue Jul 20, 2021 at 6:20 PM WEST, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Document

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Mark Brown
On Tue, Jul 20, 2021 at 11:20:25AM -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). Acked-by: Mark Brown signature.asc Description: PGP signature

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Alexandre Belloni
On 20/07/2021 11:20:25-0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Documentation/d

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Laurent Pinchart
Hi Rob, Thank you for the patch. On Tue, Jul 20, 2021 at 11:20:25AM -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, s

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Sam Ravnborg
Hi Rob, On Tue, Jul 20, 2021 at 11:20:25AM -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names

Re: [PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Mauro Carvalho Chehab
Em Tue, 20 Jul 2021 11:20:25 -0600 Rob Herring escreveu: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). > > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Docu

[PATCH] dt-bindings: Remove "status" from schema examples

2021-07-20 Thread Rob Herring
There's no reason to have "status" properties in examples. "okay" is the default, and "disabled" turns off some schema checks ('required' specifically). Enabling qca,ar71xx causes a warning, so let's fix the node names: Documentation/devicetree/bindings/net/qca,ar71xx.example.dt.yaml: phy@3: '#p