Received-SPF: pass (google.com: domain of linux-kernel+bounces-69445-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <bdfea458-6fd9-4118-aa9f-9caec2237923@intel.com>
Date: Fri, 16 Feb 2024 14:41:37 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net-next 3/3] Revert "net: bcmgenet: Ensure MDIO
 unregistration has clocks enabled"
Content-Language: en-US
To: Florian Fainelli <florian.fainelli@broadcom.com>, <netdev@vger.kernel.org>
CC: 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>, "Justin
 Chen" <justin.chen@broadcom.com>
References: <20240216184237.259954-1-florian.fainelli@broadcom.com>
 <20240216184237.259954-4-florian.fainelli@broadcom.com>
From: Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <20240216184237.259954-4-florian.fainelli@broadcom.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SGNDSXJDOFlwVGtESmlyUGZ1dVBVU00wMVBtRmlRTmhTQjJDSlhhS0VBekl0?=
 =?utf-8?B?RGp1cGs2ZXFma01ETUZHT3dSOTkwZHJJNVRFczFtcnYxbUgwREt4b213NUNN?=
 =?utf-8?B?S3VwTXMwMGZwYzVRSkwwckpVdGczWXZXRGIwNlJmNlY2c0FndGI3U2NWQTda?=
 =?utf-8?B?L0FJU0Z6RGcxMEVFQVp3QlhQMXc3R3k4bjZXajlldlExc3RMYmlqNnJuUWRs?=
 =?utf-8?B?TEZWU1owMlNNaFRWRE5jVFBEUC9XR0plNldZVjhzZVpBcnE4Q3I0VmhoV0pI?=
 =?utf-8?B?UnMrZ2pXMjlMMlBzWFdGUkFxTFNVRGtrTGdVS21MMUZJTGlTT3pHVU5RVVdY?=
 =?utf-8?B?UjFVTmVGSmJWeTlqbmRqZjB1ZTFqWkFmSE85ZVRjTHdDdDRjcnR2dk9PUmtK?=
 =?utf-8?B?Zk5XSjY4ZmlaQ3Vsb051VkNNRkd4ekxrVlJ2QTZaZ2ZkNEU1ZEJUdExEMWw0?=
 =?utf-8?B?SUxlT2xsZjdKYnZaVTBxb2x4enpzNzlWVVQ5WE4vcDFTclE2cWViblhOYko3?=
 =?utf-8?B?YWx4TEtRNjkzSUhVU0srZkd6RjhkTDc3SXpHZ05VV2JMeXp5THdOOTZnSHAw?=
 =?utf-8?B?MlF4Sk5HZmNmK3VDMkcyV25ETUxFSGNzSkpOUU9iVkFpYnZtY2JvMFU0L1Ew?=
 =?utf-8?B?cEpDMHVubjgvSTF3Qk1CeGxXS0lZNzROd2V0WDkxdC9Ic1hWRXBsL3ZUNXVW?=
 =?utf-8?B?ZGdEbXFtenFKUzlMS1VrZDV5NHkybGVLWTgxblFhcGw3NlVQNkJjSHhmdUlR?=
 =?utf-8?B?N0FZdzI4WU16eXg3bmg5elI2THdsR25INUVrNEtPNHloWC9oTi9MeGwzUDc3?=
 =?utf-8?B?T1RKc3RWRGpjV2FoMGVEeU1VdllNTE0wTm15ZDdTOC9iZVVRWmM1R1UwMXVF?=
 =?utf-8?B?RktDKzBranR1b2pYVkpHNW5xNTFhNCtlTnNEOTYxTzZyYTdsZGVjQUVkcGhQ?=
 =?utf-8?B?azZzMmtBdzR5cE1lMWJMcTdPWU01a1VZSzNCUFkzUUVybzg5OXp3TDR6dHVm?=
 =?utf-8?B?ZEoybjIxL3dFcE1vWHJuc29nKytBUk1mT0w4SksrSm1mMnZJWVJvd0lnT2Jh?=
 =?utf-8?B?WTU3VFNwMjQraG5MSVJBSHhzM0hkVnNzVGZWQTRyRHRZWHFuTXlpdGYxbDJK?=
 =?utf-8?B?a2pCQ2prckJIMXVqQ0JPRXo4M2l0MDFPZ2prWFdPSndsUmtxSXZ3UHU4MTFz?=
 =?utf-8?B?Q244QXd1eWdOTE5ENnBySHBabFJjOWFDOWxRMGtRQ2RTYnI0cEZMczZNTGkr?=
 =?utf-8?B?U0Q0L0FKMVJHb21aWnhMT2I0L1JkNlhnQ2lCVWJmazI2eDNabHVyN3JsMkkx?=
 =?utf-8?B?SmsvUUpYTytXNW5EbnpiVy9iM1pFSldrWXJVdDlUWGxKOHc0QlRsQWpDUkZW?=
 =?utf-8?B?SnNPbVl2OC95Sm5BVlh6NmJkT2JLbGlSYXM1ZnVZUXhmOFZqU1NpdFp2WE9G?=
 =?utf-8?B?c09zbUUwZU1IZGlFWjFSdi9oN3RURHJGV0lUemJoYmlBditkUXJISXFuRkNM?=
 =?utf-8?B?WkJ4dGxCcjBibExoaC85c0daR0hBTTV3TllxdWRibm1nSnlldURGcGhxS1hs?=
 =?utf-8?B?VkRvNXZPRjczNWhSMkI5K3ZKTUZiTTN2Wkt5TzR5UWpvRWNFbDdPRjJTMzJP?=
 =?utf-8?B?ZGJtUE5hVmFUY3VNd1hjTWkrMmRvZ2VoMXpoZGR1a09iL3Q5Qlc2ZlNPcFZl?=
 =?utf-8?B?ejNjQXoxRmk4dmpRZkJ1SEJuZmlqWFQ4Y3VqTWoxeFF4ZmR6RStRdDZhVWxv?=
 =?utf-8?B?VWFOSURCTG53R3pCc1BxaXU1cWg2UkNwS0ZmZVVsdnVGZFQ0NEdLeTdtRWZU?=
 =?utf-8?B?QmROUHBULzFrZGdKR28zTzhaZkxuMisxY21Ed3RTZkhNU0lXUWt6VDdyN2Rp?=
 =?utf-8?B?dzNabmpDanRBQ2xSaFlTTkNxb1Jld3dKUlNTU3VqTWFTdGRLZFphaytodmJG?=
 =?utf-8?B?TmU3WXRBTys4OG9XRHJETEFvUFhlNVFiYmJJeWh5bzdFcDFUV3p0c1FQeWZr?=
 =?utf-8?B?MzVYeHR3THR0S3U4MFhvSUY0TW80RWpEbGhxdjd3elpxc1o2QzluWlhBZ2pj?=
 =?utf-8?B?ZmJoRXdWRWJSdU4rRmhFSldicnYyUGFma3NUeDlJais5MjFCdFFKaE8rcEhh?=
 =?utf-8?B?VnBMRlp0bkkwUkY2OUxIT09WbnZma25lNmFlbVcrZ3VtK0tBSm9zUkE1eW9T?=
 =?utf-8?B?a2c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 47ccace8-1cd3-4d36-99d0-08dc2f406fd8
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Feb 2024 22:41:38.1331
 (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: hdetjvAT73rdwKeRg+j3J8XwWIFzP2d8pPciqcKeR4DnQ++5SIXmEtrtDP5wToYLTSnEDsaCmVfUmhhjZRwf2aOZIlIZYdI/e7jawDIyvuE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB7495
X-OriginatorOrg: intel.com


On 2/16/2024 10:42 AM, Florian Fainelli wrote:
> This reverts commit eac0aac07f6af47b8ba57c8064bf04ed6df73d1d ("net:
> bcmgenet: Ensure MDIO unregistration has clocks enabled"). This is no
> longer necessary now that the MDIO bus controller has a clock that it
> can manage around the I/O accesses.
> 
> Signed-off-by: Florian Fainelli <florian.fainelli@broadcom.com>
> ---

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>