From patchwork Thu Feb 18 21:09:32 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Melki X-Patchwork-Id: 384746 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER, INCLUDES_PATCH, MAILING_LIST_MULTI, MSGID_FROM_MTA_HEADER, SPF_HELO_NONE, SPF_PASS, URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 89AE5C433E0 for ; Thu, 18 Feb 2021 21:10:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 396F961493 for ; Thu, 18 Feb 2021 21:10:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230121AbhBRVKN (ORCPT ); Thu, 18 Feb 2021 16:10:13 -0500 Received: from mail-eopbgr140077.outbound.protection.outlook.com ([40.107.14.77]:21735 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S229577AbhBRVKG (ORCPT ); Thu, 18 Feb 2021 16:10:06 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b8XKEPCGSh3i63CyKL5qQH09tiIE096jSiWaNF3Zrwbz1genly46urfTVDvCUgP0RTVA8gPr7v89qzyQqUtRK+EgaPoDxJnd982WZsPuzYhfLlHWTpBXMh3jMRFZ0dNR2azBoRHSb9bBf+4Hgl/jzNPXPYg/KT92NBcNkK2hU/VHT+rOHH0e1B4ShcmMcuimSNPHzIi6ya/55AMxZ788uQADa1x7iEU7smPtlFIaFL8qAMrzkdU5TVLv8PIkzhkbR2RrqBNm1oImpvC1MNYWfMeQUBqrp0IvH0ic8MhS01t8MiFgWDhn2oHf/hFzcCnPIn9y2ecqe7NkffDfonmoaA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dkdTlMFe4y1aRSoIOb+stnA9j3rGMmf3hJmT8Tn1CoE=; b=iAlrPw0WUqwtXVHpMA0ntstwGtr73EY0lNB6NaaoFRibi18N5gVMy9Todpk8p0C1dHK28uReEDHgsrtGZ6Z9xhgYEbP6nEM/HwvXVI8vU7ZM9BHguD1oe2TN0aOWE5Tg7IigCVRoEsOVTJQj6lp4vetou+LFX3LFvta1BA6fmjk+nYvvMj41OkNC+DxSzQzP0vqyA70bLz2UzjjONJeumhrXcGrcTyr/4JHeL9F2mg89W9P7qoJohsNln5m43dOrGJcB+JmQ2MXflxJQP9Qv4vvFqm8gAEWGCc/mA46bCQDValo3GxcTkkatR+CgpKBIpw/Cf4US+msDJWfL7CRejQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=t2data.com; dmarc=pass action=none header.from=t2data.com; dkim=pass header.d=t2data.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=t2datacom.onmicrosoft.com; s=selector1-t2datacom-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dkdTlMFe4y1aRSoIOb+stnA9j3rGMmf3hJmT8Tn1CoE=; b=V/oE7d2vQGewnYEqpg7tccXDMteyCcBlY7y51r5ajlkFNQeNl1ESQ6jx0S+Fjk9NzC+a3umOzvloUO/t30JdTX/qg9XXfCWfnhciEDZr0D0FQrptalN2g001DwPcm/D4badJk0ct38nT8yaVZvywQKVZVKGjjdcyayuWa2hDWaA= Authentication-Results: vger.kernel.org; dkim=none (message not signed) header.d=none; vger.kernel.org; dmarc=none action=none header.from=t2data.com; Received: from HE1PR0602MB2858.eurprd06.prod.outlook.com (2603:10a6:3:da::10) by HE1PR0602MB3289.eurprd06.prod.outlook.com (2603:10a6:7:17::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3868.27; Thu, 18 Feb 2021 21:09:15 +0000 Received: from HE1PR0602MB2858.eurprd06.prod.outlook.com ([fe80::a40f:7a6e:fca8:eedb]) by HE1PR0602MB2858.eurprd06.prod.outlook.com ([fe80::a40f:7a6e:fca8:eedb%8]) with mapi id 15.20.3846.042; Thu, 18 Feb 2021 21:09:15 +0000 Reply-To: christian.melki@t2data.com To: netdev@vger.kernel.org From: Christian Melki Subject: [PATCH net] net: phy: micrel: set soft_reset callback to genphy_soft_reset for KSZ8081 Message-ID: <5eb8b25e-f646-ed3d-8572-9b6ef318ae9e@t2data.com> Date: Thu, 18 Feb 2021 22:09:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 Content-Language: en-US X-Originating-IP: [81.234.39.46] X-ClientProxiedBy: HE1PR09CA0061.eurprd09.prod.outlook.com (2603:10a6:7:3c::29) To HE1PR0602MB2858.eurprd06.prod.outlook.com (2603:10a6:3:da::10) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [192.168.7.217] (81.234.39.46) by HE1PR09CA0061.eurprd09.prod.outlook.com (2603:10a6:7:3c::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3868.27 via Frontend Transport; Thu, 18 Feb 2021 21:09:14 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 871c3549-e65f-4349-c134-08d8d451726e X-MS-TrafficTypeDiagnostic: HE1PR0602MB3289: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IAxXUFdzv4OG9EJBBMKu/iSW1k7PyR9fup0V+pBnYlysgIssGIjEeunFWE/XnlweYEepK8EiWhYQTBjm4JNNzzhFluW79MqYYwPQAGEVLvOaqM/wVUAmd23QMGz1N1u+WMiW6zOV4zhDaWoSbW+vcE8FShxL+SInUowFxFX3VZBhFyekAZ8ArnO8XIV9lMj9amWZnVps5Zb38HNZ3ZiDawtV1eGu5EC1907jIkGAEU2R6nDok3bZCo1AGGUXNo+ARqYHXoxOvyCjVsEOeV28ct53wLY8Oe7Fk3Bl+NEeq3nGERXRfSBcwAC6StVWPoTZ0aaoOT11vm5bhX7piUPZdKydWy+y6pAkPXni7K1QPAoodkF/mY8QTjRi/iOtEIn729ITOGh/u9Xu69XumU3K5LEeLYgZxPzCUjx5t5T9X/QExPYkdS+yfgS2kh0VO60RgOGFFO2PiKOlHmLinAHsdGl+VbPGid+VAqOJ/jajXpAFNJIDac2+aIs+LiOkjw9VAJEYYgTlZERHsvDH3wnvM5sKh0M0LQzpiXk5o6HAbfWBBUrkLESHEbGl+MEPM9leFYmywDh3YAIWnkAYyVOPtHtuu3lkfxgNxZrJc6uF5YY= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HE1PR0602MB2858.eurprd06.prod.outlook.com; PTR:; CAT:NONE; SFS:(346002)(376002)(39830400003)(396003)(136003)(366004)(42606007)(16576012)(8676002)(478600001)(31686004)(36756003)(5660300002)(2906002)(8936002)(66556008)(6916009)(186003)(66946007)(6486002)(44832011)(3450700001)(2616005)(956004)(66476007)(6666004)(26005)(52116002)(31696002)(316002)(86362001)(16526019)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData: =?utf-8?q?LtgwbGUAuWwk4q0zqNAZDnUe6yo?= =?utf-8?q?wgbAAshPvAhN/O9QXHwvsoddOH+bufSdCdZLByQCwQFHMTGtHqpRqbN5?= =?utf-8?q?tDXjXhT1jjXsWPf6J36zjXFZpMHQgBoM0NUx3pz34856zS74BktN6KUe?= =?utf-8?q?IwZF+gbLORBQVtP/kecLIDU8bwwbgPDJJSwoHe3s169FIeJHWoyRkXrH?= =?utf-8?q?9PvI+O1YgXtYmZQYqbwW2MCZw4At/YA1H6EBsGwDj79Mbyvqls9woPA3?= =?utf-8?q?URbVvwK4mkE2F0l/KOKysRBnzJ7NXRMrMtHe4wJfXop0r6dIDMFEzhUB?= =?utf-8?q?BttmUqjsuu3gQFX6kCC4TXUjXIRzk76k23ITx2D9jO0JVKdY2qvUgjki?= =?utf-8?q?7nKNcl2p9yi7T54PpKRnnQf2d5bvhVyaaPVaaDwrQgoyb1Y69EevItUN?= =?utf-8?q?lbYswAAp7KJQe/tYJIXHyj+SZUs5q1tAHg6I6YHRfNtL38k+4VL56J31?= =?utf-8?q?bnZQCMj0n2Kh2xrFEKcbxwu6RZ5r8qjl/Mv6qkva054mJHsaKtUadqoU?= =?utf-8?q?1t+y0eD6W+6MDTfADjtMhtfxzWblkZW2FclfcdLIlDQ1Vxjan4Eknkv0?= =?utf-8?q?2OLG/d40/QNhvFtzIetEpPBpIpabYuig6gDuqZFz1AHYvsCzJfZ8sSaa?= =?utf-8?q?mzFvrE9gI7sdJPBC8pYrLDfsKXXM1HWSXui/Nj0yUWH09k6U5ctZcaZ8?= =?utf-8?q?aSzrER5L7yIxdRUKU2dUaGGjT+kaNMDze1dLjuX7Nx2+apNAple49m4n?= =?utf-8?q?VljUbNY3BxNp4Q3BjwjyzhDFRjUUZPM8cvbkBPqtlmowctNPKXCq8LLB?= =?utf-8?q?QRy6/CrFLJrKVYghb2AEFOsOLPt7vklwCMnzM67+t+GD/whugn8tY4X5?= =?utf-8?q?eIbrtikA3TMyD5+w4XGeXjqC344CPqorlX+wnmXw3yo6VL5me4fnbOXH?= =?utf-8?q?FMVocLJnxAgSODsisZaYXe+erEYIVcWJ5B/p9QvA4tnO9BX0SGhxu+sf?= =?utf-8?q?ne/ObIRCS4/XeE1kQPFfdJmIjPaYU+EbuK3VQZaxSq03JsNbHVR58KlW?= =?utf-8?q?Tf4wRZYm0ad2nmHOVjAD5ZRAU+2NedQ6YdiQbzlaQ1zkVqwy9hSmqvqQ?= =?utf-8?q?GHVciojwL9M4j77JiZaqZwp+18xbaecF/eJ/9UiNQLDbHTuWNO8bwBTZ?= =?utf-8?q?xwxwzmiOolXTi6Z0al5xabvaFN9NGt4oyV5XKOmIrfIJWbWKzdUQ/gd/?= =?utf-8?q?0GcCbYLnWG+yMnz5XLn8fwbKWXKhO7vubTxoU9Hv4ObPEjLjezE5YEqw?= =?utf-8?q?WHPCC9QOI7Rn2Kff0Qyt4IUfho16GnBHKLZFxaawltEnUbLQH6ktVpN+?= =?utf-8?q?FaeDAc34FKuAJbIGMn3kAu9v9vYMF?= X-OriginatorOrg: t2data.com X-MS-Exchange-CrossTenant-Network-Message-Id: 871c3549-e65f-4349-c134-08d8d451726e X-MS-Exchange-CrossTenant-AuthSource: HE1PR0602MB2858.eurprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Feb 2021 21:09:15.1175 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 27928da5-aacd-4ba1-9566-c748a6863e6c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Q3Cgc3FhUjlqANKTwFSy7aDBOys7zfniAMoTcrKAJjeja2u889mOuUi5PyQbALxrDozafOreRINMO1Qy+9XESwdiaWwYsIjUUbVADbpj/vM= X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0602MB3289 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Following a similar reinstate for the KSZ9031. Older kernels would use the genphy_soft_reset if the PHY did not implement a .soft_reset. Bluntly removing that default may expose a lot of situations where various PHYs/board implementations won't recover on various changes. Like with implementation during a 4.9.x to 5.4.x LTS transition. I think it's a good thing to remove unwanted soft resets but wonder if it did open a can of worms? Atleast this fixes one iMX6 FEC/RMII/8081 combo. Fixes: 6e2d85ec0559 ("net: phy: Stop with excessive soft reset") Signed-off-by: Christian Melki diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c index 54e0d75203da..57f8021b70af 100644 --- a/drivers/net/phy/micrel.c +++ b/drivers/net/phy/micrel.c @@ -1295,6 +1295,7 @@ static struct phy_driver ksphy_driver[] = { .driver_data = &ksz8081_type, .probe = kszphy_probe, .config_init = ksz8081_config_init, + .soft_reset = genphy_soft_reset, .config_intr = kszphy_config_intr, .handle_interrupt = kszphy_handle_interrupt, .get_sset_count = kszphy_get_sset_count,