From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 59F71C433B4 for ; Fri, 23 Apr 2021 16:49:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 28972613C2 for ; Fri, 23 Apr 2021 16:49:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243036AbhDWQtq (ORCPT ); Fri, 23 Apr 2021 12:49:46 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:38360 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229549AbhDWQtm (ORCPT ); Fri, 23 Apr 2021 12:49:42 -0400 Received: from andrew by vps0.lunn.ch with local (Exim 4.94) (envelope-from ) id 1lZyzF-000hG0-7F; Fri, 23 Apr 2021 18:49:01 +0200 Date: Fri, 23 Apr 2021 18:49:01 +0200 From: Andrew Lunn To: Vadym Kochan Cc: "David S. Miller" , Jakub Kicinski , netdev@vger.kernel.org, Taras Chornyi , linux-kernel@vger.kernel.org, Mickey Rachamim , Vadym Kochan Subject: Re: [PATCH net-next 1/3] net: marvell: prestera: bump supported firmware version to 3.0 Message-ID: References: <20210423155933.29787-1-vadym.kochan@plvision.eu> <20210423155933.29787-2-vadym.kochan@plvision.eu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210423155933.29787-2-vadym.kochan@plvision.eu> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 23, 2021 at 06:59:31PM +0300, Vadym Kochan wrote: > From: Vadym Kochan > > New firmware version has some ABI and feature changes like: > > - LAG support > - initial L3 support > - changed events handling logic > > Signed-off-by: Vadym Kochan > --- > drivers/net/ethernet/marvell/prestera/prestera_pci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/marvell/prestera/prestera_pci.c b/drivers/net/ethernet/marvell/prestera/prestera_pci.c > index 298110119272..80fb5daf1da8 100644 > --- a/drivers/net/ethernet/marvell/prestera/prestera_pci.c > +++ b/drivers/net/ethernet/marvell/prestera/prestera_pci.c > @@ -13,7 +13,7 @@ > > #define PRESTERA_MSG_MAX_SIZE 1500 > > -#define PRESTERA_SUPP_FW_MAJ_VER 2 > +#define PRESTERA_SUPP_FW_MAJ_VER 3 > #define PRESTERA_SUPP_FW_MIN_VER 0 I could be reading the code wrong, but it looks like anybody with firmware version 2 on their machine and this new driver version results in the switch not probing? And if the switch does not probe, do they have any networking to go get the new firmware version? I think you need to provide some degree of backwards compatibly to older firmware. Support version 2 and 3. When version 4 comes out, drop support for version 2 in the driver etc. Andrew