[U-Boot] [PATCH v2 6/6] net: macb: Add compatible string for MACB on SiFive Unleashed

[Anup Patel]

On Wed, Jun 19, 2019 at 11:28 AM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Anup,
>
> On Wed, Jun 19, 2019 at 12:01 PM Anup Patel <Anup.Patel at wdc.com> wrote:
> >
> > We have a separate compatible string for MACB ethernet on SiFive
> > Unleashed so this patch adds the SiFive MACB compatible string to
> > the MACB ethernet driver.
> >
> > Signed-off-by: Anup Patel <anup.patel at wdc.com>
> > ---
> >  drivers/net/macb.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
>
> You need add the codes to handle the 10/100/1000Mbps link speed
> changes since you dropped the gemgxl clock driver.

This patch only gets U-Boot MACB driver working in 1000Mbps speed
with updated DT bindings. If we don't add the compatible string then
U-Boot MACB driver does not probed.

I totally agree with you that we need to update the U-Boot MACB
driver to support switching between 10/100/1000Mbps speeds but
I don't have bandwidth to do that right now.

I have another solution. I will drop this patch and keep both old and
new compatible string in DTB passed by OpenSBI to U-Boot. This way
current U-Boot MACB driver will be probed  and even Linux MACB
driver will work fine.

Regards,
Anup