On 05/10/2017 04:30 AM, Tom Rini wrote:
hey Tom, I am not sure how to move this forward really so let me
clarify where I think we stand:
1. The linux kernel does not need the clock property in the uart
nodes (only u-boot does: serial_pl01x.c needs fixing).
2. ehci is not present in the linux kernel poplar dts yet but it
will be eventually.
with this in mind, what is blocking the acceptance of the patchset?
I can do v5 using the linux kernel dts as is and creating a
hi3798cv200-u-boot.dtsi that simply adds the nodes above (this time
no #include required:) )
Then when ehci is added to the kernel, the ehci node can be removed
from u-boot.dtsi
And when uboot updates its pl01x.c serial driver, the uart0 node
can be removed and the u-boot.dtsi filed completely deleted.
Can you take a stab at updating the pl01x driver? Thanks!
updating pl01x is not a big deal I dont think; however this will mean
requiring a platform specific clock driver to just use the pl01x - which
will take me some time to get into uboot for my platform (and the same
might happen to other users).
- if the issue for accepting the Poplar patchset is with the dts
modification I can revert to just not using OF for pl01x like other
platforms do.
- if the issue is that we wish to enforce each new platform to commit a
clock driver then I just need to plan for it (to be honest I didn't
envision this to be the case when all I need is to enable a uart)
maintaining full compatibility with the kernel's dts comes at a price so
just making sure that this is the direction we want to take (not just a
one off where I am the lucky one :) )
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
