[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[Michal Simek]

Hi,

please pull these ARM64 DT patches to your tree.

Thanks,
Michal


The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754:

  Linux 4.2-rc1 (2015-07-05 11:01:52 -0700)

are available in the git repository at:

  https://github.com/Xilinx/linux-xlnx.git tags/zynqmp-dt-for-4.3

for you to fetch changes up to f49310dc62f68ae9f905f9a1e42224fa23867f47:

  ARM64: zynqmp: Move SPI nodes to the right location (2015-07-31
10:46:34 +0200)

----------------------------------------------------------------
arm: Xilinx ZynqMP dt patches for v4.3

- Add SATA, GPIO, CAN, SMMU, USB, SPI, I2C, watchdog and sdhci for zynqmp
- Sort nodes in dtsi

----------------------------------------------------------------
Michal Simek (10):
      ARM64: zynqmp: Use zynqmp specific compatible string for gpio
      ARM64: zynqmp: Add CANs node for platform
      ARM64: zynqmp: Add SMMU support
      ARM64: zynqmp: Add DWC3 usb support
      ARM64: zynqmp: Enable watchdog on ep108
      ARM64: zynqmp: Enable sdhci on ep108
      ARM64: zynqmp: Add eeprom memories on i2c bus
      ARM64: zynqmp: Enable spi flashes on ep108
      ARM64: zynqmp: Move uart and ttcs to the right location
      ARM64: zynqmp: Move SPI nodes to the right location

Suneel Garapati (1):
      devicetree: xilinx: zynqmp: add sata node

 arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts |  89
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 arch/arm64/boot/dts/xilinx/zynqmp.dtsi      | 233
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------
 2 files changed, 241 insertions(+), 81 deletions(-)

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150731/d5148d6b/attachment.sig>

[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[Olof Johansson]

On Fri, Jul 31, 2015 at 10:49:35AM +0200, Michal Simek wrote:
> Hi,
> 
> please pull these ARM64 DT patches to your tree.
> 
> Thanks,
> Michal
> 
> 
> The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754:
> 
>   Linux 4.2-rc1 (2015-07-05 11:01:52 -0700)
> 
> are available in the git repository at:
> 
>   https://github.com/Xilinx/linux-xlnx.git tags/zynqmp-dt-for-4.3
> 
> for you to fetch changes up to f49310dc62f68ae9f905f9a1e42224fa23867f47:
> 
>   ARM64: zynqmp: Move SPI nodes to the right location (2015-07-31
> 10:46:34 +0200)
> 
> ----------------------------------------------------------------
> arm: Xilinx ZynqMP dt patches for v4.3
> 
> - Add SATA, GPIO, CAN, SMMU, USB, SPI, I2C, watchdog and sdhci for zynqmp
> - Sort nodes in dtsi

You seem to sort these alphabetically. We normally encourage sorting by unit
address instead. Was there some reason you chose alphabetical?


-Olof

[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[Michal Simek]

Hi Olof,

On 08/05/2015 11:00 AM, Olof Johansson wrote:
> On Fri, Jul 31, 2015 at 10:49:35AM +0200, Michal Simek wrote:
>> Hi,
>>
>> please pull these ARM64 DT patches to your tree.
>>
>> Thanks,
>> Michal
>>
>>
>> The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754:
>>
>>   Linux 4.2-rc1 (2015-07-05 11:01:52 -0700)
>>
>> are available in the git repository at:
>>
>>   https://github.com/Xilinx/linux-xlnx.git tags/zynqmp-dt-for-4.3
>>
>> for you to fetch changes up to f49310dc62f68ae9f905f9a1e42224fa23867f47:
>>
>>   ARM64: zynqmp: Move SPI nodes to the right location (2015-07-31
>> 10:46:34 +0200)
>>
>> ----------------------------------------------------------------
>> arm: Xilinx ZynqMP dt patches for v4.3
>>
>> - Add SATA, GPIO, CAN, SMMU, USB, SPI, I2C, watchdog and sdhci for zynqmp
>> - Sort nodes in dtsi
> 
> You seem to sort these alphabetically. We normally encourage sorting by unit
> address instead. Was there some reason you chose alphabetical?

Normally if you look for something you can't remember addresses but you
remember names that's why this sorting is more natural at least for me.

I have checked APM/EXYNOS/Juno/SPR for example (and probably others) and
there is no order at all.
If this is any strict requirement to sort nodes by addresses I am happy
to send patch on the top of this series to fix it. That's why please let
me know.

There are other questions what to do with clocks for example - having
them on the bus like it is done for us and Cavium/Exynos/Seattle or
having separate node on the bus like APM.
Using soc node is another question.

I know that Grant wanted to update any guide which should cover these
stuff but I am not aware if we have any doc which describe it.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150805/8e2493c0/attachment.sig>

[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[Olof Johansson]

On Wed, Aug 05, 2015 at 05:03:03PM +0200, Michal Simek wrote:
> Hi Olof,
> 
> On 08/05/2015 11:00 AM, Olof Johansson wrote:
> > On Fri, Jul 31, 2015 at 10:49:35AM +0200, Michal Simek wrote:
> >> Hi,
> >>
> >> please pull these ARM64 DT patches to your tree.
> >>
> >> Thanks,
> >> Michal
> >>
> >>
> >> The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754:
> >>
> >>   Linux 4.2-rc1 (2015-07-05 11:01:52 -0700)
> >>
> >> are available in the git repository at:
> >>
> >>   https://github.com/Xilinx/linux-xlnx.git tags/zynqmp-dt-for-4.3
> >>
> >> for you to fetch changes up to f49310dc62f68ae9f905f9a1e42224fa23867f47:
> >>
> >>   ARM64: zynqmp: Move SPI nodes to the right location (2015-07-31
> >> 10:46:34 +0200)
> >>
> >> ----------------------------------------------------------------
> >> arm: Xilinx ZynqMP dt patches for v4.3
> >>
> >> - Add SATA, GPIO, CAN, SMMU, USB, SPI, I2C, watchdog and sdhci for zynqmp
> >> - Sort nodes in dtsi
> > 
> > You seem to sort these alphabetically. We normally encourage sorting by unit
> > address instead. Was there some reason you chose alphabetical?
> 
> Normally if you look for something you can't remember addresses but you
> remember names that's why this sorting is more natural at least for me.

Main purpose of sorting tends to be to avoid having add-add conflicts by new
entries going into different parts of the file. If different files are sorted
differently (but see below), it's hard for contributors to figure out where
they should add stuff.

> I have checked APM/EXYNOS/Juno/SPR for example (and probably others) and
> there is no order at all.

Yeah, it's in pretty bad shape. There have been some efforts to fix sorting in
the past, but it tends to degrade over time. I suspect part of this is due to
some patches lacking enough context such that the tools apply them to the wrong
part of the file. In other cases it's just neglect or forgetfulness.

> If this is any strict requirement to sort nodes by addresses I am happy
> to send patch on the top of this series to fix it. That's why please let
> me know.

It's not a strict requirement, it's just that when others have made efforts to
sort they have done so by unitid instead.

I'll merge this branch, it's probably not worth it to follow up with another
reshuffle. As you say, most other files are in even worse shape so it's
a losing battle.

> There are other questions what to do with clocks for example - having
> them on the bus like it is done for us and Cavium/Exynos/Seattle or
> having separate node on the bus like APM.
> Using soc node is another question.
> 
> I know that Grant wanted to update any guide which should cover these
> stuff but I am not aware if we have any doc which describe it.

Sounds like a suitable topic for one of many DT sessions at conferences around
the world these days, if you're attending any. It's usually easier to corner
people at those and hash things out.


-Olof

[GIT PULL] Xilinx ZynqMP DT changes for v4.3

[Michal Simek]

On 08/06/2015 09:32 AM, Olof Johansson wrote:
> On Wed, Aug 05, 2015 at 05:03:03PM +0200, Michal Simek wrote:
>> Hi Olof,
>>
>> On 08/05/2015 11:00 AM, Olof Johansson wrote:
>>> On Fri, Jul 31, 2015 at 10:49:35AM +0200, Michal Simek wrote:
>>>> Hi,
>>>>
>>>> please pull these ARM64 DT patches to your tree.
>>>>
>>>> Thanks,
>>>> Michal
>>>>
>>>>
>>>> The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754:
>>>>
>>>>   Linux 4.2-rc1 (2015-07-05 11:01:52 -0700)
>>>>
>>>> are available in the git repository at:
>>>>
>>>>   https://github.com/Xilinx/linux-xlnx.git tags/zynqmp-dt-for-4.3
>>>>
>>>> for you to fetch changes up to f49310dc62f68ae9f905f9a1e42224fa23867f47:
>>>>
>>>>   ARM64: zynqmp: Move SPI nodes to the right location (2015-07-31
>>>> 10:46:34 +0200)
>>>>
>>>> ----------------------------------------------------------------
>>>> arm: Xilinx ZynqMP dt patches for v4.3
>>>>
>>>> - Add SATA, GPIO, CAN, SMMU, USB, SPI, I2C, watchdog and sdhci for zynqmp
>>>> - Sort nodes in dtsi
>>>
>>> You seem to sort these alphabetically. We normally encourage sorting by unit
>>> address instead. Was there some reason you chose alphabetical?
>>
>> Normally if you look for something you can't remember addresses but you
>> remember names that's why this sorting is more natural at least for me.
> 
> Main purpose of sorting tends to be to avoid having add-add conflicts by new
> entries going into different parts of the file. If different files are sorted
> differently (but see below), it's hard for contributors to figure out where
> they should add stuff.

This should be filtered by maintainers. I wasn't too strict for Zynq
mostly for historical reasons but ZynqMP is different case because we
have more time for doing this.
It means just doesn't matter which sorting is used. It is just necessary
to keep eyes on it. Also if you have sorting based on addresses or by
names it is pretty visible for newcomers to see the style and follow it.
If there is a mess they just follow the mess.


>> I have checked APM/EXYNOS/Juno/SPR for example (and probably others) and
>> there is no order at all.
> 
> Yeah, it's in pretty bad shape. There have been some efforts to fix sorting in
> the past, but it tends to degrade over time. I suspect part of this is due to
> some patches lacking enough context such that the tools apply them to the wrong
> part of the file. In other cases it's just neglect or forgetfulness.

I would like to sort zynq too alphabetically because as I said it is
more natural for me. Will you accept these changes?


>> If this is any strict requirement to sort nodes by addresses I am happy
>> to send patch on the top of this series to fix it. That's why please let
>> me know.
> 
> It's not a strict requirement, it's just that when others have made efforts to
> sort they have done so by unitid instead.
> 
> I'll merge this branch, it's probably not worth it to follow up with another
> reshuffle. As you say, most other files are in even worse shape so it's
> a losing battle.

Thanks.

> 
>> There are other questions what to do with clocks for example - having
>> them on the bus like it is done for us and Cavium/Exynos/Seattle or
>> having separate node on the bus like APM.
>> Using soc node is another question.
>>
>> I know that Grant wanted to update any guide which should cover these
>> stuff but I am not aware if we have any doc which describe it.
> 
> Sounds like a suitable topic for one of many DT sessions at conferences around
> the world these days, if you're attending any. It's usually easier to corner
> people at those and hash things out.

I will be in valley next month and maybe also at Linaro connect and will
probably go to KS. Definitely cleanup up this mess is interesting topic
for me.
I think that will be the best what we have done at KS at Edinburgh to
have ARM specific session to target wider audience and cleanup these
things. People tend to copy each other and if we have mess in the main
repo then this mess is just propagated down.

Another thing what we have found recently in zynq reset patch is syscon
usage in connection to using child nodes if you should explicitely
reference syscon or add dependency based on position that syscon is
parent all the time.

Definitely all these DT things should be discussed in person between arm
platform maintainers and having session somewhere will be great.

BTW: have 2 more small patches for adding reset controller. Can I send
another PR or wait for one more release?

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150806/b0d5fdc1/attachment.sig>