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

Alex Marginean alexm.osslist at gmail.com
Wed Jul 24 13:05:45 UTC 2019


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.

Thanks!
Alex

> 
> 
> 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
> 



More information about the U-Boot mailing list