Date:   Mon, 20 Mar 2023 17:36:23 +0100
From:   Michal Kubiak <michal.kubiak@intel.com>
To:     Christian Marangi <ansuelsmth@gmail.com>
CC:     Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        "Paolo Abeni" <pabeni@redhat.com>,
        Rob Herring <robh+dt@kernel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        Gregory Clement <gregory.clement@bootlin.com>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        Andy Gross <agross@kernel.org>,
        "Bjorn Andersson" <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Pavel Machek <pavel@ucw.cz>, Lee Jones <lee@kernel.org>,
        John Crispin <john@phrozen.org>, <netdev@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-arm-msm@vger.kernel.org>, <linux-leds@vger.kernel.org>
Subject: Re: [net-next PATCH v5 06/15] net: phy: phy_device: Call into the
 PHY driver to set LED brightness
Message-ID: <ZBiLh3RMRdVKnXCT@localhost.localdomain>
References: <20230319191814.22067-1-ansuelsmth@gmail.com>
 <20230319191814.22067-7-ansuelsmth@gmail.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20230319191814.22067-7-ansuelsmth@gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sPBX/K6b6nj6Pb5Rl4AH744nPGcjnVHdu48xNBMEtq1rQQ4+W5YwRb5ufwF1?=
 =?us-ascii?Q?0ZQWXLshhVRlpKfzw46z9ktWUeb4sKzhxTIkzwGM4GGHILhDF9jEkpku87Tr?=
 =?us-ascii?Q?sgbdmO9wcu/QjUC9fCOqLRsNB86ogSXRZXgIhD6DDdHfTnChTuBUccPih92b?=
 =?us-ascii?Q?9ogJ80VrV+wFJSWq2HMn/f6GJVUdG/bT6dW8O/iw8lHegSJo9PaIP68r/I3C?=
 =?us-ascii?Q?PSAhlUx/Bp/DTBNTR6lOcpIm7yscDkqrtb/enq8RyrfDbEXSjfVFVvuGWJD4?=
 =?us-ascii?Q?Oth7i/UtNd71YZF6bMtQR01WKv67jB2VMdXq4eQZTaaSH43d/t4hNTFlitCt?=
 =?us-ascii?Q?z8D7WzRDCVi/R8yX5HWlvozajKYTyZPi1PqYSYsXEp7JxwedU8UTRwkHkB4o?=
 =?us-ascii?Q?9fx0E9jt7RqwIhdeLHCeXJsuZ33MU1JsrBXiWH1NptDTDGyNOFmLbRvA11vS?=
 =?us-ascii?Q?GQ/cDSe37IvZHR5qM38s0gQms67GYDGZbzaBpolNITYNhoMutkk9FKXXvaUh?=
 =?us-ascii?Q?JbkGqFMIsijxg/RRhrPitMkbeKaTvQy+dsdakA0AjCvhXl0bfXBOEEpvv/Hb?=
 =?us-ascii?Q?GHojIVCLZD4aP6eTZTzxaeYV067EnNqalbaydrGGo+q9bQSJ+TPKn8OAgT0N?=
 =?us-ascii?Q?C2movGLh5+LS6uaYoE7kcxJyvGddH9WD5GX8xK7iwQOEEMnP6tZq6kdvn9od?=
 =?us-ascii?Q?qeHfkrW9J0RXM0BPiOgAKSf70/fT4bMgQLElw6x+++JSQSyK1sfLCeWvwoE3?=
 =?us-ascii?Q?rQC3EIEBVwTs4G0LeMtZigWSseGcp7GO1x1kYnl7vYJvNeV13Qw4xEBJTgc0?=
 =?us-ascii?Q?U8IN0In5KSbzRA2pdTiJAC2pIb7aH3yFbaLE7Fkd1YDEO7YkzBckvShT3vmU?=
 =?us-ascii?Q?HUi5UKY4bMvk+HRkuOB9/9JXsJQ/8JDA262NuF/cCzZRXoZF0dXFfShH0HU9?=
 =?us-ascii?Q?NMI5YmAIiYAvuULaWeY8ivBIE+Mv2UQivxc9TmT96AKL35ZQz++iMtdKzwx9?=
 =?us-ascii?Q?gzYUoYgEDHtOVcnHn+eGyWFfKuS7nbOlkKA4Z5rxDYq+CBS6Jl1IngfqZ6EA?=
 =?us-ascii?Q?rQnzQ9eWcmML0QsPw3wzLi8gO+ckU6R0MezeSfAaPXGb4itVlLMJFuWqENdH?=
 =?us-ascii?Q?I+tAH6D4GPoLKdqUlvg5+WtP8lPU/xIDSaFOy7Gxon7sNvVt0knOGAj0WrB/?=
 =?us-ascii?Q?ZRuchXRxYIl+uF2ABQ4sx15vMr3VRMtAnddGvzuyMGPGZ5B2hUY60uvRoolW?=
 =?us-ascii?Q?hxBxD/KdwFxv96z6pJZaaOy3pLps0AzRb/iTKYFyAtECPNJAoojE2NZFRF+l?=
 =?us-ascii?Q?wWqcZXDc3fm86XhooCaVZT2jrtymSbY1yBZmNU3u+HMuJK1BCjq7q5cpNQTH?=
 =?us-ascii?Q?U2GuyIXHi6gwdKyuzEhREHrpehNFhnLZsd1pp5e3tigyCit8ZKRRy+7WBMZb?=
 =?us-ascii?Q?6XHYurSOvl1wobaJH7w1vGdsijqHGcwfRncmltZz0977MEpeIiixmkyvGskE?=
 =?us-ascii?Q?SGqJ1Nb4oMzNFtOJGf50mzQINQjtQ7ikGmuVxsuq1L15MUhdUFKDkrGoWo8J?=
 =?us-ascii?Q?klh1qXMYbbfI7LN4m/UyU6AdVQNrOOsaG0Su7mcGmMteJ5pqiNQw+LaHnHH9?=
 =?us-ascii?Q?jA=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 21811197-3e0f-4220-6211-08db296143ba
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB2937.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Mar 2023 16:36:33.1036
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 45/4pCBO+zy+OWLaDfQyIywNpg2CY4bVwBs0bLPH+0MpNnvxzpNFrpRfeny5ai5zcbNTaOThdMLGVRYyXEspqQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4625
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sun, Mar 19, 2023 at 08:18:05PM +0100, Christian Marangi wrote:
> From: Andrew Lunn <andrew@lunn.ch>
> 
> Linux LEDs can be software controlled via the brightness file in /sys.
> LED drivers need to implement a brightness_set function which the core
> will call. Implement an intermediary in phy_device, which will call
> into the phy driver if it implements the necessary function.
> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>