mbox series

[v4,0/3] dtc bus and unit address checks

Message ID 20170320144418.5021-1-robh@kernel.org
Headers show
Series dtc bus and unit address checks | expand

Message

Rob Herring March 20, 2017, 2:44 p.m. UTC 
This is a series of checks designed to check problems commonly found in
binding reviews. The first patch adds checks for PCI bridges and devices.
The 2nd patch is new in this version and adds checks for simple-bus. The
3rd patch is a default check if bus type is not set to check for '0x' or
leading 0s in unit addresses.

Rob

Rob Herring (3):
  checks: Add bus checks for PCI buses
  checks: Add bus checks for simple-bus buses
  checks: Warn on node name unit-addresses with '0x' or leading 0s

 checks.c                       | 231 +++++++++++++++++++++++++++++++++++++++++
 dtc.h                          |   5 +
 tests/run_tests.sh             |   2 +
 tests/unit-addr-leading-0s.dts |  12 +++
 tests/unit-addr-leading-0x.dts |  12 +++
 5 files changed, 262 insertions(+)
 create mode 100644 tests/unit-addr-leading-0s.dts
 create mode 100644 tests/unit-addr-leading-0x.dts

-- 
2.10.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

David Gibson March 21, 2017, 3:50 a.m. UTC | #1 
On Mon, Mar 20, 2017 at 09:44:15AM -0500, Rob Herring wrote:
> This is a series of checks designed to check problems commonly found in

> binding reviews. The first patch adds checks for PCI bridges and devices.

> The 2nd patch is new in this version and adds checks for simple-bus. The

> 3rd patch is a default check if bus type is not set to check for '0x' or

> leading 0s in unit addresses.

> 

> Rob

> 

> Rob Herring (3):

>   checks: Add bus checks for PCI buses

>   checks: Add bus checks for simple-bus buses

>   checks: Warn on node name unit-addresses with '0x' or leading 0s


Applied to master, thanks.

> 

>  checks.c                       | 231 +++++++++++++++++++++++++++++++++++++++++

>  dtc.h                          |   5 +

>  tests/run_tests.sh             |   2 +

>  tests/unit-addr-leading-0s.dts |  12 +++

>  tests/unit-addr-leading-0x.dts |  12 +++

>  5 files changed, 262 insertions(+)

>  create mode 100644 tests/unit-addr-leading-0s.dts

>  create mode 100644 tests/unit-addr-leading-0x.dts

> 


-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson