Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1281311 > unrolled thread

Re: [PATCH] Enable PCIe PHY support in Cygnus

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] Enable PCIe PHY support in Cygnus Florian Fainelli <f.fainelli@gmail.com> - 2015-12-02 00:20 +0100
    Re: [PATCH] Enable PCIe PHY support in Cygnus Ray Jui <rjui@broadcom.com> - 2015-12-02 00:20 +0100

#1281311 — Re: [PATCH] Enable PCIe PHY support in Cygnus

On 24/11/15 16:12, Florian Fainelli wrote:
> On 18/11/15 10:16, Ray Jui wrote:
>> This patch enales PCIe PHY in device tree for Broadcom Cygnus based platforms
>>
>> This patch is developed based on v4.4-rc1 and available here:
>> https://github.com/Broadcom/cygnus-linux/tree/cygnus-pcie-phy-dt-v1
>>
>> Ray Jui (1):
>>   ARM: dts: enable PCIe PHY support for Cygnus
> 
> Applied to devicetree/next with Scott's Acked-by, thanks!

This caused the DTC compiler to warn:

Warning (reg_format): "reg" property in /axi/phy@0301d0a0/phy@0 has
invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
Warning (reg_format): "reg" property in /axi/phy@0301d0a0/phy@1 has
invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
Warning (avoid_default_addr_size): Relying on default #address-cells
value for /axi/phy@0301d0a0/phy@0
Warning (avoid_default_addr_size): Relying on default #size-cells value
for /axi/phy@0301d0a0/phy@0
Warning (avoid_default_addr_size): Relying on default #address-cells
value for /axi/phy@0301d0a0/phy@1
Warning (avoid_default_addr_size): Relying on default #size-cells value
for /axi/phy@0301d0a0/phy@1
  CC      drivers/base/power/runtime.o
  DTC     arch/arm/boot/dts/bcm911360k.dtb
  CC      lib/bitmap.o

I added an #address-cells = <0> and #size-cells = <1> to fix this, since
your reg property is a single digit.
-- 
Florian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1281319

On 12/1/2015 3:12 PM, Florian Fainelli wrote:
> On 24/11/15 16:12, Florian Fainelli wrote:
>> On 18/11/15 10:16, Ray Jui wrote:
>>> This patch enales PCIe PHY in device tree for Broadcom Cygnus based platforms
>>>
>>> This patch is developed based on v4.4-rc1 and available here:
>>> https://github.com/Broadcom/cygnus-linux/tree/cygnus-pcie-phy-dt-v1
>>>
>>> Ray Jui (1):
>>>    ARM: dts: enable PCIe PHY support for Cygnus
>>
>> Applied to devicetree/next with Scott's Acked-by, thanks!
>
> This caused the DTC compiler to warn:
>
> Warning (reg_format): "reg" property in /axi/phy@0301d0a0/phy@0 has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> Warning (reg_format): "reg" property in /axi/phy@0301d0a0/phy@1 has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> Warning (avoid_default_addr_size): Relying on default #address-cells
> value for /axi/phy@0301d0a0/phy@0
> Warning (avoid_default_addr_size): Relying on default #size-cells value
> for /axi/phy@0301d0a0/phy@0
> Warning (avoid_default_addr_size): Relying on default #address-cells
> value for /axi/phy@0301d0a0/phy@1
> Warning (avoid_default_addr_size): Relying on default #size-cells value
> for /axi/phy@0301d0a0/phy@1
>    CC      drivers/base/power/runtime.o
>    DTC     arch/arm/boot/dts/bcm911360k.dtb
>    CC      lib/bitmap.o
>
> I added an #address-cells = <0> and #size-cells = <1> to fix this, since
> your reg property is a single digit.
>

Sorry I missed that. Both are required properties.

Thanks!

Ray
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web