All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew@lunn.ch>
To: Vadym Kochan <vadym.kochan@plvision.eu>
Cc: linux-firmware@kernel.org, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Oleksandr Mazur <oleksandr.mazur@plvision.eu>,
	Serhiy Boiko <serhiy.boiko@plvision.eu>,
	Serhiy Pshyk <serhiy.pshyk@plvision.eu>,
	Volodymyr Mytnyk <volodymyr.mytnyk@plvision.eu>,
	Taras Chornyi <taras.chornyi@plvision.eu>,
	Mickey Rachamim <mickeyr@marvell.com>
Subject: Re: [GIT PULL] linux-firmware: mrvl: prestera: Update Marvell Prestera Switchdev v3.0 with policer support
Date: Thu, 17 Jun 2021 18:45:14 +0200	[thread overview]
Message-ID: <YMt8GvxSen6gB7y+@lunn.ch> (raw)
In-Reply-To: <20210617154206.GA17555@plvision.eu>

On Thu, Jun 17, 2021 at 06:42:06PM +0300, Vadym Kochan wrote:
> The following changes since commit 0f66b74b6267fce66395316308d88b0535aa3df2:
> 
>   cypress: update firmware for cyw54591 pcie (2021-06-09 07:12:02 -0400)
> 
> are available in the Git repository at:
> 
>   https://github.com/PLVision/linux-firmware.git mrvl-prestera
> 
> for you to fetch changes up to a43d95a48b8e8167e21fb72429d860c7961c2e32:
> 
>   mrvl: prestera: Update Marvell Prestera Switchdev v3.0 with policer support (2021-06-17 18:22:57 +0300)
> 
> ----------------------------------------------------------------
> Vadym Kochan (1):
>       mrvl: prestera: Update Marvell Prestera Switchdev v3.0 with policer support
> 
>  mrvl/prestera/mvsw_prestera_fw-v3.0.img | Bin 13721584 -> 13721676 bytes
>  1 file changed, 0 insertions(+), 0 deletions(-)

Hi Vadym

You keep the version the same, but add new features? So what does the
version number actually mean? How does the driver know if should not
use the policer if it cannot tell old version 3.0 from new version
3.0?  How is a user supposed to know if they have old version 3.0
rather than new 3.0, when policer fails?

    Andrew

  reply	other threads:[~2021-06-17 16:45 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-17 15:42 [GIT PULL] linux-firmware: mrvl: prestera: Update Marvell Prestera Switchdev v3.0 with policer support Vadym Kochan
2021-06-17 16:45 ` Andrew Lunn [this message]
2021-06-17 16:58   ` Vadym Kochan
2021-06-18  1:18     ` Andrew Lunn
2021-06-18  9:58       ` Vadym Kochan
2021-06-18 13:41         ` Andrew Lunn
2021-06-22 11:44           ` Vadym Kochan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YMt8GvxSen6gB7y+@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=linux-firmware@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mickeyr@marvell.com \
    --cc=netdev@vger.kernel.org \
    --cc=oleksandr.mazur@plvision.eu \
    --cc=serhiy.boiko@plvision.eu \
    --cc=serhiy.pshyk@plvision.eu \
    --cc=taras.chornyi@plvision.eu \
    --cc=vadym.kochan@plvision.eu \
    --cc=volodymyr.mytnyk@plvision.eu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.