On 04/09/2020 14:40, Stefan Roese wrote:
Hi Andre,

On 04.09.20 14:35, Andre Heider wrote:

<snip>

First I though that you have been syncing the file with the Linux kernel
version. But now I see that its sync'ed with downstream U-Boot most
likely. As for the license of the file: The Linux kernel version has
this SPDX tag:

// SPDX-License-Identifier: (GPL-2.0+ OR MIT)

Changing this to this tag would haven been fine (AFAICT). But using a
diffent one from a downstream U-Boot repository is a bit troublesome
for my taste.

Can't you sync with Linux dts/dtsi files instead at some point?

at some point, yes. As far as I understand that's not yet possible due to the comphy driver. Quoting Marek's cover letter "[PATCH u-boot-marvell 00/11] Armada 37xx: port comphy to generic-phys (PLEASE TEST)" from 19th April:

 > I have ported the COMPHY and UTMI PHY drivers from Linux to U-Boot.
 > With this we are able to abandon the current comphy_a3700 driver, which
 > is incompatible with Linux' device trees. So if we want to have DTS
 > files for A3720 boards identical to Linux', we have to do this.

So unfortunately we can't use Linux' dts files for a3700 yet. Marek said he plans to work on that set again in the near future, so hopefully that lands rather sooner than later ;)

Ah, thanks. I remember now. ;)

Can you then please drop the sync with the downstream dts and better
only make the necessary changes for eMMC? In a git bisectable way
please?

Sure, I've just sent v2, completely redone. That also prepares synching from Linux, actually already imports two dts from from there.

Regards,
Andre

Reply via email to