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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4762DFA373D for ; Fri, 21 Oct 2022 16:26:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230037AbiJUQ0o (ORCPT ); Fri, 21 Oct 2022 12:26:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52992 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229872AbiJUQ03 (ORCPT ); Fri, 21 Oct 2022 12:26:29 -0400 Received: from vps0.lunn.ch (unknown [156.67.10.101]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E2FDE2764D2; Fri, 21 Oct 2022 09:26:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=iq1eYfHdRkn3kCl6a44vE/x3fUxXovvRR4vp6IbnJdg=; b=Ap0GIMnfBBlNRXdHy7XSru8ijN lN5XXWP0RyKhQ7s4p7vP9JrsBk8TiKeMH8DBKP7/Y4CRfLIJYzXd1IsMpg4QBzbRhiyacOHEL3Bmr ZNuU8ED6Ohs6CmCDofagncJLq6/p1wJGwwnIL9/5SJ0LJRy0fI397ror/6yZADXZhnks=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1oluAC-000FKg-Ca; Fri, 21 Oct 2022 17:42:24 +0200 Date: Fri, 21 Oct 2022 17:42:24 +0200 From: Andrew Lunn To: "Russell King (Oracle)" Cc: "David S. Miller" , Jakub Kicinski , devicetree@vger.kernel.org, Eric Dumazet , Heiner Kallweit , Krzysztof Kozlowski , netdev@vger.kernel.org, Paolo Abeni , Rob Herring Subject: Re: [PATCH net-next 0/7] net: sfp: improve high power module implementation Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, Oct 19, 2022 at 02:28:20PM +0100, Russell King (Oracle) wrote: > Hi, > > This series aims to improve the power level switching between standard > level 1 and the higher power levels. > > The first patch updates the DT binding documentation to include the > minimum and default of 1W, which is the base level that every SFP cage > must support. Hence, it makes sense to document this in the binding. > > The second patch enforces a minimum of 1W when parsing the firmware > description, and optimises the code for that case; there's no need to > check for SFF8472 compliance since we will not need to touch the > A2h registers. > > Patch 3 validates that the module supports SFF-8472 rev 10.2 before > checking for power level 2 - rev 10.2 is where support for power > levels was introduced, so if the module doesn't support this revision, > it doesn't support power levels. Setting the power level 2 declaration > bit is likely to be spurious. Or it is yet another case of violating the standard. The bit is valid, the revision is wrong in the EEPROM. How long do you think it will be before we see a quirk like this? > Patch 4 does the same for power level 3, except this was introduced in > SFF-8472 rev 11.9. The revision code was never updated, so we use the > rev 11.4 to signify this. Great, the standard itself is broken. Andrew