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:   Wed, 10 May 2023 12:03:15 +0200
From:   Simon Horman <simon.horman@corigine.com>
To:     Yan Wang <rk.code@outlook.com>
Cc:     andrew@lunn.ch, hkallweit1@gmail.com, davem@davemloft.net,
        edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux@armlinux.org.uk
Subject: Re: [PATCH v3] net: mdiobus: Add a function to deassert reset
Message-ID: <ZFtr47e5Q9oZ73yq@corigine.com>
References: <KL1PR01MB5448A33A549CDAD7D68945B9E6779@KL1PR01MB5448.apcprd01.prod.exchangelabs.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <KL1PR01MB5448A33A549CDAD7D68945B9E6779@KL1PR01MB5448.apcprd01.prod.exchangelabs.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?GadyNb3K4bL9qSR+YgaTj/YdWSYnGePZscjmZ0tyZrtXWmEzzWRcBiXvuBs9?=
 =?us-ascii?Q?sfW+0ZOFpS8bwVY54kHb1tv2N7RKqcTEMjiAvRRI5w/lloN+WdVM2G6ir+iv?=
 =?us-ascii?Q?4OYHw6tHu1NgY+ixG2KgcHFnojTfKeu+utiHRgB024ML7V6rQq6w/7nZtZO0?=
 =?us-ascii?Q?TSZVxuBd10iR/jJL1uhVNL0x+5HlXLJJPlOx7hNPtggilWV4hqTod/MKGiXC?=
 =?us-ascii?Q?NF1v5sWZlZNf40jat5zCBx1sHOKI3a7S/Xrgv9I4ASDTVaefHemwVg1T8Juc?=
 =?us-ascii?Q?OSoMDfCfHvVS+u0HnQimGPe+WII4gqt7MSUcc9z0qgTT86YNQ0Xjueb1a0gT?=
 =?us-ascii?Q?lH96CiaX7Vj4IotLshG06Plg/XTzZbcSegyGR+vKKwNVGjJhwARuxymQ1e4Y?=
 =?us-ascii?Q?huFmUyn6qdBehH6Tuo2ShsX5c2OplUC6mbq++mmf+udYOiEVu/8KQdqbE0wn?=
 =?us-ascii?Q?Mck/pY3vcFO+UlpzV4VdOxnP9//R4ibCY0krdWVQpgiOYD8V/o8pgWo3tcQk?=
 =?us-ascii?Q?MlNwOeVp+2jTNRXui7eeQ42A4X7CLPQ/CshStLuLGoPRglR9giTYpk8x3iQo?=
 =?us-ascii?Q?FY0sHcWBclluH5ihZlxtDeLASwB2gXQIvyKO9R2NfXNciyKY8VI2g0Nwt5mY?=
 =?us-ascii?Q?MVEGf4FEgir7mI2qrP9x+EBz+gMLOCTlAsvQGFcR5MEu/gauKBCnA/6YfQyO?=
 =?us-ascii?Q?xkGCNCEJGz3qSwapJAsnjgZrETjtmY2MJzJE1eB5FSpKxAfCHQohBBXvVhej?=
 =?us-ascii?Q?ey2RvzxZhTtnBHV5XMb20bQI/0O+ui3AV5AK4cs+YUHwwuxQ3FmqONuESny5?=
 =?us-ascii?Q?i+xTBV/VJe1txjSmBY6/bRRAbolTXBS+O6YBdPMGRCR+juaTmzKnTs0Jyg/P?=
 =?us-ascii?Q?acjqBdRgsy5tEUxru0SMGiLH4F06XgtnV3E/7FnJaNYTNHvSTzEXFtawadvL?=
 =?us-ascii?Q?5Me9Iot/LlNVXUtDrQPYc5eepb8ZBJCZlChZZgBAW+xh4hKtgT7CRqUfJCfx?=
 =?us-ascii?Q?IqytnLhIzv6jwUy5fh2PMrPeOGj1dB9DByMJM7w0FL9CIeR91rSF2F5n1jnl?=
 =?us-ascii?Q?iivhzfnfWzLCjKgGf3+CXn+Ek36J2iwdDjpe9lXbj48KFbx//v3DXSpnMixD?=
 =?us-ascii?Q?dQ1H968COTDW/Q2oxWVuP2VAHq2PgdasZEyRroChT7eMV/IA1Xy3rbAu9Mmv?=
 =?us-ascii?Q?Q0Us6P5YtoTEhgPnYCA+LAxR4Jcj2s96EpElThrwa8h6g4Tullc+2xDJWsPU?=
 =?us-ascii?Q?MQv21fP0sndaJqffLiUI/+GNBY0gxukUblVu17z20OsBTDUw7Fkxro1AHUoH?=
 =?us-ascii?Q?TiXqRg4hNdDSFsMYkonuXdSIaxhmXKoENAXMjqEb3VnAuNOkOcGiur4QmXOh?=
 =?us-ascii?Q?3UPQGtROM1OISaPRJGdUlmOZFVGk+zoP3M0cm5Br0GU7bTCI1jTVwKE+AfNK?=
 =?us-ascii?Q?/09yN8khzGbEJkBPnjU/l640F4TM852v8Xb+Mesho2WvmmsWguf+dpxJ/+Rd?=
 =?us-ascii?Q?24GXsZk3/gUTQwoT613Mx4xZAXVYzSoup9TSOjVruj9AGKGYLr9YhOIu9AS6?=
 =?us-ascii?Q?tHezDF41hYBaChpEuin0xT1toZDB1uagH9RhMQ7rCcIJ34C2L2qB4OZe4Fy4?=
 =?us-ascii?Q?G+KZbQndj/Lzy9rsQWQb67lVENXWhk+gFQpg9oWds+SZU4JxFpv/1m5JPJVp?=
 =?us-ascii?Q?cYjhgw=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b0b3f243-8fc7-4240-71dd-08db513dcabc
X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2023 10:03:24.1445
 (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: 0XUXARiDQEJCn3bRcmG4UEOOvwCyC17ALVz99OHF6Fk3XeSmqgAIqprKq0A+a5zTUQ/MvIoWkjvJ8vdQhjN/JrNbaNLp8v1VKHHYOxlD0cY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR13MB5590
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, May 10, 2023 at 04:15:22PM +0800, Yan Wang wrote:
> It is possible to mount multiple sub-devices on the mido bus.
> The hardware power-on does not necessarily reset these devices.
> The device may be in an uncertain state, causing the device's ID
> to not be scanned.
> 
> So, before adding a reset to the scan, make sure the device is in
> normal working mode.
> 
> I found that the subsequent drive registers the reset pin into the
> structure of the sub-device to prevent conflicts, so release the
> reset pin.
> 
> Signed-off-by: Yan Wang <rk.code@outlook.com>

Hi Yan,

v3 was posted less than 15 minutes after v2.
Please wait 24h between posting patches to give reviewers an opportunity
to review patches.

Link: https://kernel.org/doc/html/v6.1/process/maintainer-netdev.html