From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 12 May 2020 10:25:03 -0400 Subject: [PATCH v1 5/7] arm: dts: s700: add node for ethernet controller In-Reply-To: <17165fd0-fd21-8262-91e7-b8c347818e86@arm.com> References: <1589034315-19722-1-git-send-email-amittomer25@gmail.com> <1589034315-19722-6-git-send-email-amittomer25@gmail.com> <17165fd0-fd21-8262-91e7-b8c347818e86@arm.com> Message-ID: <20200512142503.GA12564@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: