Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 10:58:35 -0700 Roman Bacik wrote: > Thank you very much for your suggestion. To simplify, we will decouple > this issue from the current bnxt driver submission. Very well, that seems reasonable. I am willing to help you with this afterwards. Marek

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
chuchardt > ; Kory Maincent ; > Michal Simek ; Patrick Delaunay > ; Peng Fan ; Priyanka > Jain ; Rayagonda Kokatanur > ; Sean Anderson > > Subject: Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands > > On Wed, 27 Oct 2021 10:02:41 -0700 > Roman Bacik wrote: > > &g

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 10:02:41 -0700 Roman Bacik wrote: > Marek, Simon, > > Thank you very much for your comments. We will remove bnxt commands and will > probe bnxt driver each boot in the next version. > Thanks, Roman I think that the idea of not loading fw or initializing the controller

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
...@lists.denx.de>; Bharat Gooty ; > > Aswath Govindraju ; Bin Meng > > ; Franck LENORMAND > > ; Heinrich Schuchardt > > ; Kory Maincent ; > > Michal Simek ; Patrick Delaunay > > ; Peng Fan ; > Priyanka > > Jain ; Rayagonda Kokatanur > > ; Se

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
chuchardt > ; Kory Maincent ; > Michal Simek ; Patrick Delaunay > ; Peng Fan ; Priyanka > Jain ; Rayagonda Kokatanur > ; Sean Anderson > > Subject: Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands > > On Wed, 27 Oct 2021 08:05:11 -0700 > Roman Bacik wrote: > > > c

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Marek Behún
On Wed, 27 Oct 2021 08:05:11 -0700 Roman Bacik wrote: > chimp_ld_secure #this command loads FW, which is necessary for PCIe to > enumerate it > pci enum #this command is necessary to call bnxt_bind Wait, so what is this firmware for? Is it firmware for the netXtreme controller or for PCIe

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
chuchardt > ; Kory Maincent ; > Michal Simek ; Patrick Delaunay > ; Peng Fan ; Priyanka > Jain ; Rayagonda Kokatanur > ; Sean Anderson > > Subject: Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands > > Hi Roman, > > On Wed, 27 Oct 2021 at 09:05, Roman Bacik > wrot

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Simon Glass
ooty ; > > Aswath Govindraju ; Bin Meng > > ; Franck LENORMAND > > ; Heinrich Schuchardt > > ; Kory Maincent ; > > Michal Simek ; Patrick Delaunay > > ; Peng Fan ; Priyanka > > Jain ; Rayagonda Kokatanur > > ; Sean Anderson > > > > S

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-27 Thread Roman Bacik
Kory Maincent ; > Michal Simek ; Patrick Delaunay > ; Peng Fan ; Priyanka > Jain ; Rayagonda Kokatanur > ; Sean Anderson > > Subject: Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands > > On Tue, 26 Oct 2021 09:02:54 -0700 > Roman Bacik wrote: > > > On Tue, Oct

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Roman Bacik
On Tue, Oct 26, 2021 at 9:52 AM Marek Behún wrote: > > On Tue, 26 Oct 2021 09:40:44 -0700 > Roman Bacik wrote: > > > On Tue, Oct 26, 2021 at 9:02 AM Roman Bacik > > wrote: > > > > > > On Tue, Oct 26, 2021 at 8:55 AM Marek Behún wrote: > > > > > > > > On Tue, 26 Oct 2021 08:14:28 -0700 > > > >

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Marek Behún
On Tue, 26 Oct 2021 09:40:44 -0700 Roman Bacik wrote: > On Tue, Oct 26, 2021 at 9:02 AM Roman Bacik wrote: > > > > On Tue, Oct 26, 2021 at 8:55 AM Marek Behún wrote: > > > > > > On Tue, 26 Oct 2021 08:14:28 -0700 > > > Roman Bacik wrote: > > > > > > > Hi Marek, > > > > > > > > We do not want

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Marek Behún
On Tue, 26 Oct 2021 09:02:54 -0700 Roman Bacik wrote: > On Tue, Oct 26, 2021 at 8:55 AM Marek Behún wrote: > > > > On Tue, 26 Oct 2021 08:14:28 -0700 > > Roman Bacik wrote: > > > > > Hi Marek, > > > > > > We do not want this driver to be automatically probed. It is not needed > > > all the

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Roman Bacik
On Tue, Oct 26, 2021 at 9:02 AM Roman Bacik wrote: > > On Tue, Oct 26, 2021 at 8:55 AM Marek Behún wrote: > > > > On Tue, 26 Oct 2021 08:14:28 -0700 > > Roman Bacik wrote: > > > > > Hi Marek, > > > > > > We do not want this driver to be automatically probed. It is not needed > > > all the time

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Roman Bacik
On Tue, Oct 26, 2021 at 8:55 AM Marek Behún wrote: > > On Tue, 26 Oct 2021 08:14:28 -0700 > Roman Bacik wrote: > > > Hi Marek, > > > > We do not want this driver to be automatically probed. It is not needed > > all the time and also slows down the boot time. We have stripped down > > everything

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Marek Behún
On Tue, 26 Oct 2021 08:14:28 -0700 Roman Bacik wrote: > Hi Marek, > > We do not want this driver to be automatically probed. It is not needed > all the time and also slows down the boot time. We have stripped down > everything else to bare minimum. > Thanks, > > Roman Hi Roman, OK, that is

RE: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Roman Bacik
; > Michal Simek ; Patrick Delaunay > ; Peng Fan ; Priyanka > Jain ; Rayagonda Kokatanur > ; Sean Anderson > ; Simon Glass > Subject: Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands > > On Mon, 25 Oct 2021 16:44:44 -0700 > Roman Bacik wrote: > > > From: Bharat Gooty

Re: [PATCH v3 2/2] cmd: brcm: netXtreme commands

2021-10-26 Thread Marek Behún
On Mon, 25 Oct 2021 16:44:44 -0700 Roman Bacik wrote: > From: Bharat Gooty > > Following netXtreme commands are supported: > probe, remove. > > Signed-off-by: Bharat Gooty > > Signed-off-by: Roman Bacik Hi Roman, why do you need to have custom command for probing / removing the driver?

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

2021-10-25 Thread Roman Bacik
From: Bharat Gooty Following netXtreme commands are supported: probe, remove. Signed-off-by: Bharat Gooty Signed-off-by: Roman Bacik --- Changes in v3: - remove commands set/get mac/speed - add doc/bnxt.rst cmd/Kconfig | 2 + cmd/broadcom/Kconfig | 10 +