Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-31 Thread David Miller
From: Or Gerlitz Date: Tue, 30 May 2017 23:32:22 +0300 > On Sun, May 28, 2017 at 10:26 AM, Yotam Gigi wrote: >> On 05/23/2017 06:38 PM, David Miller wrote: >>> From: Yotam Gigi >>> Date: Tue, 23 May 2017 18:14:15 +0300 >

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-30 Thread Or Gerlitz
On Sun, May 28, 2017 at 10:26 AM, Yotam Gigi wrote: > On 05/23/2017 06:38 PM, David Miller wrote: >> From: Yotam Gigi >> Date: Tue, 23 May 2017 18:14:15 +0300 >>> Sorry, I am not sure I understand. You think that drivers should not >>> implement >>>

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-29 Thread Jakub Kicinski
[swapping Ben's email] On Mon, 29 May 2017 11:52:27 +0300, Or Gerlitz wrote: > On Mon, May 29, 2017 at 3:17 AM, Jakub Kicinski wrote: > > On Sun, 28 May 2017 10:26:49 +0300, Yotam Gigi wrote: > > >> This problem is even more relevant in the Mellanox HCA driver team, which >

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-29 Thread Or Gerlitz
On Mon, May 29, 2017 at 3:17 AM, Jakub Kicinski wrote: > On Sun, 28 May 2017 10:26:49 +0300, Yotam Gigi wrote: >> This problem is even more relevant in the Mellanox HCA driver team, which >> would >> like to use that code in order to burn the HCA firmware, but not intend to >>

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-29 Thread Yotam Gigi
On 05/29/2017 03:17 AM, Jakub Kicinski wrote: > On Sun, 28 May 2017 10:26:49 +0300, Yotam Gigi wrote: >> On 05/23/2017 06:38 PM, David Miller wrote: >>> From: Yotam Gigi >>> Date: Tue, 23 May 2017 18:14:15 +0300 >>> Sorry, I am not sure I understand. You think that

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-28 Thread Jakub Kicinski
On Sun, 28 May 2017 10:26:49 +0300, Yotam Gigi wrote: > On 05/23/2017 06:38 PM, David Miller wrote: > > From: Yotam Gigi > > Date: Tue, 23 May 2017 18:14:15 +0300 > > > >> Sorry, I am not sure I understand. You think that drivers should not > >> implement > >> ethtool's

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-28 Thread Yotam Gigi
On 05/23/2017 06:38 PM, David Miller wrote: > From: Yotam Gigi > Date: Tue, 23 May 2017 18:14:15 +0300 > >> Sorry, I am not sure I understand. You think that drivers should not >> implement >> ethtool's flash_device callback anymore? do you have an alternative for >>

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Jiri Pirko
Tue, May 23, 2017 at 05:38:59PM CEST, da...@davemloft.net wrote: >From: Yotam Gigi >Date: Tue, 23 May 2017 18:14:15 +0300 > >> Sorry, I am not sure I understand. You think that drivers should not >> implement >> ethtool's flash_device callback anymore? do you have an

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread David Miller
From: Yotam Gigi Date: Tue, 23 May 2017 18:14:15 +0300 > Sorry, I am not sure I understand. You think that drivers should not implement > ethtool's flash_device callback anymore? do you have an alternative for > firmware > flash? As stated, export an MTD device.

RE: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Mintz, Yuval
> Patches 6-8 add the "ethtool -f" and the boot-time firmware upgrade > on > >> the > mlxsw spectrum driver. > >>> When we tried using `ethtool -E' for qed we got burned for trying to > >>> use > >> the magic > >>> value [1]. When we suggested extending it to allow some private data

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Yotam Gigi
On 05/23/2017 06:04 PM, Mintz, Yuval wrote: Patches 6-8 add the "ethtool -f" and the boot-time firmware upgrade on >> the mlxsw spectrum driver. >>> When we tried using `ethtool -E' for qed we got burned for trying to use >> the magic >>> value [1]. When we suggested extending it to

RE: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Mintz, Yuval
> >> Patches 6-8 add the "ethtool -f" and the boot-time firmware upgrade on > the > >> mlxsw spectrum driver. > > When we tried using `ethtool -E' for qed we got burned for trying to use > the magic > > value [1]. When we suggested extending it to allow some private data > indications, > > Ben

Re: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Yotam Gigi
On 05/23/2017 04:18 PM, Mintz, Yuval wrote: >> Add support for device firmware flash on mlxsw spectrum. The firmware files >> are expected to be in the Mellanox Firmware Archive version 2 (MFA2) >> format. >> >> The firmware flash can be triggered via "ethtool -f" or on driver >> initialization

RE: [patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Mintz, Yuval
> Add support for device firmware flash on mlxsw spectrum. The firmware files > are expected to be in the Mellanox Firmware Archive version 2 (MFA2) > format. > > The firmware flash can be triggered via "ethtool -f" or on driver > initialization > time if the device firmware version does not

[patch net-next 0/9] mlxsw: Support firmware flash

2017-05-23 Thread Jiri Pirko
From: Jiri Pirko Add support for device firmware flash on mlxsw spectrum. The firmware files are expected to be in the Mellanox Firmware Archive version 2 (MFA2) format. The firmware flash can be triggered via "ethtool -f" or on driver initialization time if the device