On 12/05/2020 15:25, Tom Rini wrote:
On Tue, May 12, 2020 at 03:18:33PM +0100, André Przywara wrote:
On 09/05/2020 15:25, Amit Singh Tomar wrote:
This patch adds node for ethernet controller found on Action Semi OWL S700 SoC.
Since, there is no upstream Linux binding exist for S700 ethernet controller, Changes are put in u-boot specific dtsi file.
But that should not be the S700 SoC .dtsi, instead the cubieboard .dts file, since you specify the PHY mode in here (which is board specific).
The general way to move forward here is that bindings should be getting proposed to Linux and accepted there, and U-Boot can take a WIP of them, that gets updated later on to match, but we shouldn't get it here first.
Yes, this is of course a stop-gap measure, but a useful one: Being able to TFTP a kernel helps with developing the kernel port, especially since U-Boot doesn't support MMC (yet). And there are easier things than pushing a DT binding into the kernel tree without having a Linux driver ready ...
Cheers, Andre