[U-Boot] [PATCH 4/4] arm: dts: Set custom names for cp110 master/slave MDIO buses

[Joe Hershberger]

On Wed, Jul 24, 2019 at 8:06 AM Alex Marginean <alexm.osslist at gmail.com> wrote:
>
> On 7/23/2019 8:22 PM, Nevo Hed wrote:
> > For context from upstream linux,
> > https://github.com/torvalds/linux/commit/f66b2aff46eaf0825dbe1560e28fcb845dd2a215
> >
> > I was intending to submit the parallel change here once fully tested
> > end-end on our hardware,
> > but if not too late my proposal is to name the above nodes as
> > "cpm-mdio" & "cps-mdio"
>
> I'm OK with "-mdio", I can send a v2.
> Please node that I used a property called 'device-name' rather than
> 'mdio-name', I'm not sure if you want to pick that up too or not.

Let me know when this is settled and I'll take the series.

Thanks,
-Joe

> >
> >
> > On Tue, Jul 23, 2019 at 1:03 PM Nevo Hed <nhed+uboot at starry.com> wrote:
> >>
> >> Sorry for the late input
> >>
> >> The cp110s have an mdio and an xmdio so
> >> Locally I have been naming them cpm-xmdio, cpm-mdio, cps-xmdio, cps-mdio
> >>
> >> I knew the new field is coming but didn't catch that the 'dtsi's patch
> >>
> >> [also apologies if you get this a second time]
> >>
> >> On Mon, Jul 22, 2019 at 8:30 PM Joe Hershberger <joe.hershberger at ni.com> wrote:
> >>>
> >>> On Wed, Jul 17, 2019 at 10:12 AM Alex Marginean
> >>> <alexandru.marginean at nxp.com> wrote:
> >>>>
> >>>> Implicitly Marvell MDIO driver uses DT node names for devices, but in this
> >>>> case that is not unique.  Set MDIO device names for master/slave to
> >>>> cpm/cps.
> >>>>
> >>>> Signed-off-by: Alex Marginean <alexm.osslist at gmail.com>
> >>>
> >>> Acked-by: Joe Hershberger <joe.hershberger at ni.com>
> >>> _______________________________________________
> >>> U-Boot mailing list
> >>> U-Boot at lists.denx.de
> >>> https://lists.denx.de/listinfo/u-boot
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > https://lists.denx.de/listinfo/u-boot
> >
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot