[U-Boot] x86: cherryhill: Fix DTC warning

Message ID 1528858874-7073-1-git-send-email-bmeng.cn@gmail.com
State Accepted
Delegated to: Bin Meng
Headers show
Series
  • [U-Boot] x86: cherryhill: Fix DTC warning
Related show

Commit Message

Bin Meng June 13, 2018, 3:01 a.m.
Fix warning when compiling cherryhill.dts with latest DTC:

  "Warning (avoid_unnecessary_addr_size): /pci/pch@1f,0: unnecessary
   #address-cells/#size-cells without "ranges" or child "reg" property"

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---

 arch/x86/dts/cherryhill.dts | 2 --
 1 file changed, 2 deletions(-)

Comments

Simon Glass June 14, 2018, 3:11 p.m. | #1
On 12 June 2018 at 21:01, Bin Meng <bmeng.cn@gmail.com> wrote:
> Fix warning when compiling cherryhill.dts with latest DTC:
>
>   "Warning (avoid_unnecessary_addr_size): /pci/pch@1f,0: unnecessary
>    #address-cells/#size-cells without "ranges" or child "reg" property"
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
>  arch/x86/dts/cherryhill.dts | 2 --
>  1 file changed, 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Bin Meng June 15, 2018, 10:03 a.m. | #2
On Thu, Jun 14, 2018 at 11:11 PM, Simon Glass <sjg@chromium.org> wrote:
> On 12 June 2018 at 21:01, Bin Meng <bmeng.cn@gmail.com> wrote:
>> Fix warning when compiling cherryhill.dts with latest DTC:
>>
>>   "Warning (avoid_unnecessary_addr_size): /pci/pch@1f,0: unnecessary
>>    #address-cells/#size-cells without "ranges" or child "reg" property"
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>> ---
>>
>>  arch/x86/dts/cherryhill.dts | 2 --
>>  1 file changed, 2 deletions(-)
>
> Reviewed-by: Simon Glass <sjg@chromium.org>

applied to u-boot-x86, thanks!

Patch

diff --git a/arch/x86/dts/cherryhill.dts b/arch/x86/dts/cherryhill.dts
index c3a6aad..3e29683 100644
--- a/arch/x86/dts/cherryhill.dts
+++ b/arch/x86/dts/cherryhill.dts
@@ -75,8 +75,6 @@ 
 		pch@1f,0 {
 			reg = <0x0000f800 0 0 0 0>;
 			compatible = "intel,pch9";
-			#address-cells = <1>;
-			#size-cells = <1>;
 
 			irq-router {
 				compatible = "intel,irq-router";