Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 8 Jun 2023 13:37:27 +0200
From:   Simon Horman <simon.horman@corigine.com>
To:     Florian Fainelli <florian.fainelli@broadcom.com>
Cc:     netdev@vger.kernel.org, Doug Berger <opendmb@gmail.com>,
        Broadcom internal kernel review list 
        <bcm-kernel-feedback-list@broadcom.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] net: phy: broadcom: Add support for setting LED
 brightness
Message-ID: <ZIG9d+nT64f93F8Y@corigine.com>
References: <20230607183453.2587726-1-florian.fainelli@broadcom.com>
 <20230607183453.2587726-3-florian.fainelli@broadcom.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230607183453.2587726-3-florian.fainelli@broadcom.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xGzPJBHWdc1hjj7xnvcLwtNLLp4pfdeNPGk4PAJDPl5qTjhz7arM486g8aRQ?=
 =?us-ascii?Q?j4spOtsGWMTvFO6U0hkpUACgfgXCF+kHrvHPQg+1aPIEYFRHIj9DZGRbfnXi?=
 =?us-ascii?Q?qBgUTb3du891021ZhC+AlCfteWcUH/u5h63wRhvG5z6WJDr08Hx+84m62vyu?=
 =?us-ascii?Q?hiL3ZnVcXnsQ5Xesw45MByFfl6d4SZnh1GhCGDqS4UC516r8rOFh5+5J1CeJ?=
 =?us-ascii?Q?nqCI+gQc/ja7UbIXVYssyMg5LG+ZRv5biqY1fr/iN6s2u1rLvyGuKTAKNXbh?=
 =?us-ascii?Q?OcZwjwgvzhvhLzLJvM0bSXH6Y1tdXFV/TBMV7zhMSkEX7ijHuVKwpb3r6cL7?=
 =?us-ascii?Q?U61MjdOOHZwkRobWPc1fJVGEiqEmQml8PprdmG7P029Ib+wNxKqwhAZERVyf?=
 =?us-ascii?Q?ACREWABqnmfY5iho75omw4GViY5FiM5SwhAnnYUhREJhPhsVnSBzoyiIY1EX?=
 =?us-ascii?Q?O0IBxOzOnFnRtsjvIQdo9pNSAOnurut3tomNVKUd7+xs34vs+ghZAY9OSuDV?=
 =?us-ascii?Q?nRJGrRptYVLsN3ep1tRHDP54HgD3DCnSuCQ9M8BkksE5ld6waqjPw6I8g0m4?=
 =?us-ascii?Q?yZwhPOsIZuaJODxDDy9IU1yOH4pkovw/7ILADlNocCr3A5lkqk+4uLPsUk5H?=
 =?us-ascii?Q?8YYfvRk0fO8V1tJh52pnjCvgSn3h8+oH6pynlG2RXSOcOGBC3D66NlbzPDfW?=
 =?us-ascii?Q?KENXKEiHt8/e6xsioOJaXtwGvBTlJUF0InKp7BXXmT6vnWBlSTE9I5UOnFah?=
 =?us-ascii?Q?MvEG9hWvphlhwPlUGUM1whfDqGFN2OUzhbZT+bYqRMXH80463ZG2k/P0gqUe?=
 =?us-ascii?Q?M0xj1JMGH0knpJ2U/sT0ItNpo/GY8GbGXfNhcqNNy19Xcvq0DUjEcvkmN6JE?=
 =?us-ascii?Q?sQvLe3fYHiIvIpP3lBHAr79/+HuIbPqBVVphZ1AHh61DMg2gv+g5aEV6em67?=
 =?us-ascii?Q?0iO2gmN75KLl3QWVQXtCqCE/ARcWWBj6CyMns1C6mwrUL3JflP/7nJfopSvV?=
 =?us-ascii?Q?wTqBYqUgVtz9zYih8KG/E4JWShEEkhm9lc5sk51/6b0kmLP8vdxkhJBQdihK?=
 =?us-ascii?Q?2XXSW2Ci6etSjj0pAJZ+a6f18P+T6levr5LyuS2toohYq8cEAcmboS1Tvmrx?=
 =?us-ascii?Q?pXqyhIvfC8JXLuacc0zvmxFplAva4NsnY5MQLq5a6+zSEqi5IQdI8DF5D7aT?=
 =?us-ascii?Q?JBp0UrndfC7PTFSRU+Ga6xkHxSeCjsWhg7SUjVOuu8YQM3kYwNOvZL7Gz0/u?=
 =?us-ascii?Q?VvKC5U9ohbBoJ6Gv9F2fKBKztQrX4+93YD7UrtD6jeQwP/5khUUka2ROr7f9?=
 =?us-ascii?Q?WN66rebHwgwaJUpa3Glzv5g84KRMHJIL6UsXrlEtGJKUacmOmEg9YBXcEeyS?=
 =?us-ascii?Q?PZs73/IF3F6SnvnybJptHbdcekCoOup4IqbJKtfq3bmCrx45AT+sJOLfHP0B?=
 =?us-ascii?Q?fp59fAA1NKf5YccnbttvfU3zdotYz9J5maVE9GMEcWoVVGh7m26QHcRZVsu0?=
 =?us-ascii?Q?k1dNmzQe+YjBZ6pfxyci3pbgzsMLOrVoIFDeZY+AAh/3l5hxfCNMRtffVtsc?=
 =?us-ascii?Q?CETE5DZrcWlAZbr356kgMC+lsB+Klgy8Hncdws61SSduwFyzKfSGoWDPyfvW?=
 =?us-ascii?Q?vCDAx4fBRNub5NXy82ELwv/jp4L+0Sth5S6z6bu+BDq+ZVH+alvTqiVOfpC6?=
 =?us-ascii?Q?yzewyg=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7ab073cd-eb0a-4ebe-c322-08db6814c0ab
X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 11:37:34.5521
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xsPAWtNFh8Ax/y0sMQ55GLwgbhUwENcGFg3zf1oLDlFpxmdAxOiAiuVOr/DjKWfdACfwno9gZvaP21bwJTQHTRWnAMCtyGkXRgLz3UafUFA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR13MB3623
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Jun 07, 2023 at 11:34:53AM -0700, Florian Fainelli wrote:
> Broadcom PHYs have two LEDs selector registers which allow us to control
> the LED assignment, including how to turn them on/off.
> 
> Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>

Reviewed-by: Simon Horman <simon.horman@corigine.com>