lkml.org 
[lkml]   [2021]   [Aug]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 4/6] ARM: dts: aspeed: Add Facebook Cloudripper (AST2600) BMC
On Fri, Aug 13, 2021 at 01:29:17AM +0000, Joel Stanley wrote:
> On Thu, 5 Aug 2021 at 22:28, <rentao.bupt@gmail.com> wrote:
> > +&mdio1 {
> > + status = "okay";
>
> You're enabling this but it looks like it's unused?

Thanks Joel for the careful review. The MDIO controller is not paired
with BMC MAC; instead, it's connected to the MDC/MDIO interface of the
on-board switch (whose ports are connected to BMC, Host and front panel
management port).

In other word, the MDIO controller is used, but we do need some user
space scripts to control the controller. What's your suggestion? For
example, should I add some comments to explain the purpose?


Cheers,

Tao

>
> > +};
> > +
> > +&mdio3 {
> > + status = "okay";
> > +
> > + ethphy1: ethernet-phy@13 {
> > + compatible = "ethernet-phy-ieee802.3-c22";
> > + reg = <0x0d>;
> > + };
> > +};
> > +
> > +&mac3 {
> > + status = "okay";
> > + phy-mode = "rgmii";
> > + phy-handle = <&ethphy1>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_rgmii4_default>;
> > +};

\
 
 \ /
  Last update: 2021-08-13 05:41    [W:0.071 / U:0.176 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site