diff mbox series

[2/6] dt-bindings: mfd: google,cros-ec: explicitly allow additional properties

Message ID 20201013160845.1772-3-thunder.leizhen@huawei.com
State New
Headers show
Series fix all errors except one reported by dt_binding_check | expand

Commit Message

Zhen Lei Oct. 13, 2020, 4:08 p.m. UTC
There are so many properties have not been described in this yaml file,
and a lot of errors will be reported. Especially, some yaml files such as
google,cros-ec-typec.yaml, extcon-usbc-cros-ec.yaml can not pass the
self-check, because of the examples. So temporarily allow additional
properties to keep the comprehensive dt_binding_check result clean.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>

---
 Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.8.3

Comments

Dan Murphy Oct. 13, 2020, 5:53 p.m. UTC | #1
Zhen

On 10/13/20 11:08 AM, Zhen Lei wrote:
> There are so many properties have not been described in this yaml file,

> and a lot of errors will be reported. Especially, some yaml files such as

> google,cros-ec-typec.yaml, extcon-usbc-cros-ec.yaml can not pass the

> self-check, because of the examples. So temporarily allow additional

> properties to keep the comprehensive dt_binding_check result clean.


My preference is to fix the binding to pass the checks and not just work 
around the issues. Working around the issues may mean the issues never 
get fixed.

Dan
Zhen Lei Oct. 15, 2020, 6:55 a.m. UTC | #2
On 2020/10/14 21:38, Rob Herring wrote:
> On Wed, Oct 14, 2020 at 12:08:41AM +0800, Zhen Lei wrote:

>> There are so many properties have not been described in this yaml file,

>> and a lot of errors will be reported. Especially, some yaml files such as

>> google,cros-ec-typec.yaml, extcon-usbc-cros-ec.yaml can not pass the

>> self-check, because of the examples. So temporarily allow additional

>> properties to keep the comprehensive dt_binding_check result clean.

>>

>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>

>> ---

>>  Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-

>>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> There's proper fixes for these under review.


That's a good news.

> 

> Rob

> 

> .

>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
index f49c0d5d31ad20e..e18047583a0c8fa 100644
--- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
+++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
@@ -87,7 +87,7 @@  then:
     google,cros-ec-spi-msg-delay: false
     spi-max-frequency: false
 
-additionalProperties: false
+additionalProperties: true
 
 examples:
   # Example for I2C