[PATCH v2 2/2] cmd: brcm: netXtreme commands

[Roman Bacik]

> -----Original Message-----
> From: Marek Behún <kabel at kernel.org>
> Sent: Monday, October 25, 2021 7:01 AM
> To: Roman Bacik <roman.bacik at broadcom.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Bharat Gooty
> <bharat.gooty at broadcom.com>; Bin Meng <bmeng.cn at gmail.com>; Franck
> LENORMAND <franck.lenormand at nxp.com>; Heinrich Schuchardt
> <xypron.glpk at gmx.de>; Kory Maincent <kory.maincent at bootlin.com>;
> Michal Simek <michal.simek at xilinx.com>; Patrick Delaunay
> <patrick.delaunay at foss.st.com>; Peng Fan <peng.fan at nxp.com>; Priyanka
> Jain <priyanka.jain at nxp.com>; Rayagonda Kokatanur
> <rayagonda.kokatanur at broadcom.com>; Sean Anderson
> <sean.anderson at seco.com>; Simon Glass <sjg at chromium.org>
> Subject: Re: [PATCH v2 2/2] cmd: brcm: netXtreme commands
>
> On Fri, 22 Oct 2021 16:23:33 -0700
> Roman Bacik <roman.bacik at broadcom.com> wrote:
>
> > From: Bharat Gooty <bharat.gooty at broadcom.com>
> >
> > Following netXtreme commands are supported:-
> > Device probe, remove, supported speeds, get/set speeds and
> > get/set MAC address.
>
> NAK.
>
> - "bnxt <bnxt_eth#> get mac
>
>   U-Boot uses the ethaddr and ethNaddr environment variables for MAC
>   addresses. You don't need a new custom command for that when there is
>   a generic mechanism for this.
>
> - "<bnxt_eth#> probe" / "<bnxt_eth#> remove"
>
>   You also shouldn't need a command for driver probe / remove. DM should
>   probe your driver automatically. And if you need it for debugging,
>   please add such subcommand to the dm command.
>
> - "bnxt <bnxt_eth#> get supported_speed"
>   "bnxt <bnxt_eth#> get link_speed"
>
>   These should be available via the mdio command when you register your
>   PHY driver via appropriate API.
>
> Marek

Hi Marek,

Thank you very much for your feedback. We have two Ethernet drivers. One
is 10/100/1000 MB rmii driver, which is being used as you have described.
The second one is this 10/100 GB bnxt driver, which is probed and managed
on demand with these netXtreme commands. We will have a look and address
your comments.
Thanks,

Roman

-- 
This electronic communication and the information and any files transmitted 
with it, or attached to it, are confidential and are intended solely for 
the use of the individual or entity to whom it is addressed and may contain 
information that is confidential, legally privileged, protected by privacy 
laws, or otherwise restricted from disclosure to anyone else. If you are 
not the intended recipient or the person responsible for delivering the 
e-mail to the intended recipient, you are hereby notified that any use, 
copying, distributing, dissemination, forwarding, printing, or copying of 
this e-mail is strictly prohibited. If you received this e-mail in error, 
please return the e-mail to the sender, delete it from your computer, and 
destroy any printed copy of it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4206 bytes
Desc: S/MIME Cryptographic Signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20211025/c32193d5/attachment.bin>