mbox series

[0/8] ARM: dts: keystone*: Stage 1 cleanup for W=1

Message ID 20171215132102.435-1-nm@ti.com
Headers show
Series ARM: dts: keystone*: Stage 1 cleanup for W=1 | expand

Message

Nishanth Menon Dec. 15, 2017, 1:20 p.m. UTC
Hi,
Series to squash some of the build warnings with dts. At least try and get
through 2/3rd of the warnings low hanging ones with this series.

 Warning status:
As of master: http://pastebin.ubuntu.com/26188581/
After this series: http://pastebin.ubuntu.com/26188578/
Diff: 243 warnings squashed.

 + Additional patch in https://patchwork.kernel.org/patch/10114913/

Nishanth Menon (8):
  ARM: dts: keystone*: Standardize license with SPDX tag
  ARM: dts: keystone*: Use a single soc0 instance
  ARM: dts: keystone: Get rid of usage of skeleton.dtsi
  ARM: dts: keystone: Add missing unit name to interrupt controller
  ARM: dts: keystone-clocks: Add missing unit name to  clock nodes that
    have regs
  ARM: dts: keystone-hk-clocks: Add missing unit name to  clock nodes
    that have regs
  ARM: dts: keystone-k2e-clocks: Add missing unit name to  clock nodes
    that have regs
  ARM: dts: keystone-k2l-clocks: Add missing unit name to clock nodes
    that have regs

 arch/arm/boot/dts/keystone-clocks.dtsi      | 59 ++++++++++-----------
 arch/arm/boot/dts/keystone-k2e-clocks.dtsi  | 15 +++---
 arch/arm/boot/dts/keystone-k2e-evm.dts      | 11 ++--
 arch/arm/boot/dts/keystone-k2e-netcp.dtsi   |  7 +--
 arch/arm/boot/dts/keystone-k2e.dtsi         | 11 ++--
 arch/arm/boot/dts/keystone-k2g-evm.dts      | 12 +----
 arch/arm/boot/dts/keystone-k2g-ice.dts      |  3 +-
 arch/arm/boot/dts/keystone-k2g.dtsi         | 14 ++---
 arch/arm/boot/dts/keystone-k2hk-clocks.dtsi | 81 ++++++++++++++---------------
 arch/arm/boot/dts/keystone-k2hk-evm.dts     | 57 ++++++++++----------
 arch/arm/boot/dts/keystone-k2hk-netcp.dtsi  |  7 +--
 arch/arm/boot/dts/keystone-k2hk.dtsi        | 11 ++--
 arch/arm/boot/dts/keystone-k2l-clocks.dtsi  | 51 +++++++++---------
 arch/arm/boot/dts/keystone-k2l-evm.dts      | 11 ++--
 arch/arm/boot/dts/keystone-k2l-netcp.dtsi   |  7 +--
 arch/arm/boot/dts/keystone-k2l.dtsi         | 11 ++--
 arch/arm/boot/dts/keystone.dtsi             | 18 +++----
 17 files changed, 164 insertions(+), 222 deletions(-)

-- 
2.14.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

Nishanth Menon Dec. 15, 2017, 1:38 p.m. UTC | #1
Crap.. couple of typos crept in. Apologies - Santosh, if you dont want 
to manualy change, I can rebase and repost if you like to any branch 
of your choice.

On 12/15/2017 07:20 AM, Nishanth Menon wrote:
> Provide and soc0 node and reference the same to simplify dts. This also

            ^^ - should have been a
> resolves the following warnings when built with W=1:

> arch/arm/boot/dts/keystone-k2hk-evm.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name

> arch/arm/boot/dts/keystone-k2l-evm.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name

> arch/arm/boot/dts/keystone-k2e-evm.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name

> arch/arm/boot/dts/keystone-k2g-evm.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name

> arch/arm/boot/dts/keystone-k2g-ice.dtb: Warning (unit_address_vs_reg): Node /soc has a reg or ranges property, but no unit name

> 

> NOTE: Though we can reformat files by reducing 1 level of indent due to

> the use of soc0 phandle, we ommit that change to prevent un-necessary

                               ^^ -> omit
> churn in code base.

> 

> Reported-by: Rob Herring <robh@kernel.org>

> Signed-off-by: Nishanth Menon <nm@ti.com>



Apologies once again on my oversight.

-- 
Regards,
Nishanth Menon
--
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
Santosh Shilimkar Dec. 16, 2017, 8:33 a.m. UTC | #2
On 12/15/17 5:38 AM, Nishanth Menon wrote:
> Crap.. couple of typos crept in. Apologies - Santosh, if you dont want 

> to manualy change, I can rebase and repost if you like to any branch of 

> your choice.

> 

Just post it against for_4.16/keystone-dts
--
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
Santosh Shilimkar Dec. 16, 2017, 10:39 p.m. UTC | #3
On 12/16/17 12:40 PM, Nishanth Menon wrote:
> Hi,

> The following changes have been done in the updated series:

> * Updates for couple few typo errors in commit messages

> * copyrights are now always behind description

> * rebased to maintainer branch (was previously against master)

> 

> Rebased to:

>   git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git

>   for_4.16/keystone-dts : 7fbec17465fc ARM: dts: k2g-evm: Enable UART 2

> 

Thanks Nishant. Applied and should show up in next soon.

Regards,
Santosh
--
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