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;
Received-SPF: Pass (protection.outlook.com: domain of seco.com designates
 20.160.56.82 as permitted sender) receiver=protection.outlook.com;
 client-ip=20.160.56.82; helo=inpost-eu.tmcas.trendmicro.com; pr=C
Authentication-Results-Original: dkim=none (message not signed)
 header.d=none;dmarc=none action=none header.from=seco.com;
From:   Sean Anderson <sean.anderson@seco.com>
To:     Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Vinod Koul <vkoul@kernel.org>,
        Kishon Vijay Abraham I <kishon@kernel.org>,
        Michal Simek <michal.simek@amd.com>,
        linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org
Cc:     linux-arm-kernel@lists.infradead.org,
        Sean Anderson <sean.anderson@seco.com>
Subject: [PATCH] phy: zynqmp: Allow variation in refclk rate
Date:   Tue, 11 Jul 2023 15:45:39 -0400
Message-Id: <20230711194542.898230-1-sean.anderson@seco.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-Microsoft-Antispam-Message-Info-Original: vPqdVf2NVDIjqj8rcViXGvrwLfFh5tCCK9RFe6Xw/ZWNegk4YazBIx/xeim3OBAQ8OSXYmY7VqlhpOyVMyZe8Uwb8OpLp1JbpWSUjuwxQQCqTX5ZTmj0Vow0uBxjLtQjOrDJGNMqYp4s+9NevWLjbsz+6qvJEdNo6FA6hYafgg1ZYU1stSMEcfvqNkqa7tkXGuQJ+CHTbLwEM0SsbDksciAuZIeJZSxKKYkyxuDo/h4NyLNwDx3Gz/m5xhSZPYNUoPhZy3xauAjhYDoH21WLyUvw9cyyChn15dBcwS9PC5AJ7Nwzc3lsgp4sCfMdJ4tg3jaVIgNDzdpntbF/XeN+9qT6ajgB6zRgTGmVDzH2dgb594mBzz2+3S42dKSjwFnlUQDsI6SGo7lZ7lX1CETW5BY/dfJqaTgCeWeCU9RwRX5zVuW3tJRWTBOZivUyIvjVaqnc08W8aUhSlOvwTzB29/3Xge0589uF7GlNa8spslIILl60YwESvKmDUbfXrc9HF3Dz9fPXnMNDo0uTEuv2b7cRBhzbaLgpW6iO74jMWLUCFRZFCaJb2cpXIFDke81flbn+fBoQwCOI7nt0YwbzLjqsy82vSc+YfE+yCCDvXcaDQxAJhC+HrnEk64FGSuX+vp9TC8gds78/cjnt67ETVnz8r+9ylIgP1oVB83Wq4Kc=
X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB9PR03MB8847.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(396003)(136003)(376002)(366004)(39850400004)(346002)(451199021)(38350700002)(86362001)(38100700002)(186003)(6486002)(52116002)(110136005)(6666004)(36756003)(107886003)(1076003)(26005)(6506007)(6512007)(2616005)(5660300002)(2906002)(316002)(66556008)(66946007)(4744005)(66476007)(8676002)(44832011)(8936002)(83380400001)(4326008)(478600001)(41300700001);DIR:OUT;SFP:1101;
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR03MB9471
X-EOPAttributedMessage: 0
X-MS-Exchange-Transport-CrossTenantHeadersStripped: AM6EUR05FT034.eop-eur05.prod.protection.outlook.com
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id-Prvs: a71277f5-e2af-449e-4924-08db82477194
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: i6j6NXcT/XdmJok3Wc6uhjIPE0y/eaD9gxrI1UK4tTTZ2VfzuX02h7ooI3xnVTWKFVUx8H5y80h6AXWZBlVT1AulwAFJFrqQXuNTEd1qLGIVLo0lr2ytV7S+6wXMuPgfvLPNBWQGAnBf5MyXpK8MwhW2XZXXlv38yFfCAnX6LLCeGSCVqzSH+uQk8AtAjmR0Gl9Mp1Bi0p6SVZimcKm9aurLAod8T6rQpU3Vp16xqfQ8ueCE+6JeM31wzrtb9u8Ph0jRT8nPKdukPjKqNY71dm1AMYvR9Jomt2mHS9ERtUSxYYbf2Nxczk2Fqoos4DgNrqveylNKJDMWue1pjnKdc5m+bhHfFMxAwcHx0YUP/aH3fyp2xsL0nmwfYWLns0oNKZG7bQUgF0uADB9D7lJUeLCddcvWJT63V5x+1ae6LyIC0qoIsuwVk9ARTSpj+TgdPzG27n9I+0oQ16e9KnBBIF6x1S+E/lGaQjKyZYkVi8UeQr01zwFdQn9Jwr0Uh/ZHQnEMKpxuuUwvedg1LOw0+ST4eWLsFncvymbjzd8svLO553Piq1FLBL7gKgW6qdiFltnvfXLGX2zRhIeBacHga3vnq7cj1a2mEnkWGyYgJFTWM4uRWE6qY3nS5XA0ETQs2cg6F9cps/7o2U/PTw7WQ6Zbi4GcxuClImZ9xOs5jxghCOhnY1Pe39+YKSWCZ0n/rUbIvHB6Cy4aA5gwt8abt3zWDgd5Q9L1q/9Ig6FpiG4enK6jHhYpojRgoXFtp3X1TvVXftEZznQx+ypgLcATyQ==
X-Forefront-Antispam-Report: CIP:20.160.56.82;CTRY:NL;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:inpost-eu.tmcas.trendmicro.com;PTR:inpost-eu.tmcas.trendmicro.com;CAT:NONE;SFS:(13230028)(136003)(39850400004)(396003)(346002)(376002)(451199021)(40470700004)(36840700001)(46966006)(110136005)(6666004)(6486002)(82740400003)(40460700003)(7596003)(7636003)(478600001)(356005)(8676002)(5660300002)(8936002)(36756003)(2906002)(4744005)(86362001)(82310400005)(44832011)(4326008)(316002)(70206006)(70586007)(40480700001)(83380400001)(336012)(47076005)(41300700001)(1076003)(186003)(34070700002)(26005)(6506007)(2616005)(107886003)(6512007)(36860700001);DIR:OUT;SFP:1101;
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jul 2023 19:46:07.6718
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: bf94cd24-c0e9-4165-f060-08db82477861
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=bebe97c3-6438-442e-ade3-ff17aa50e733;Ip=[20.160.56.82];Helo=[inpost-eu.tmcas.trendmicro.com]
X-MS-Exchange-CrossTenant-AuthSource: AM6EUR05FT034.eop-eur05.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR03MB7331
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED 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

Due to limited available frequency ratios, the reference clock rate may
not be exactly the same as the required rate. Allow a small (100 ppm)
deviation.

Signed-off-by: Sean Anderson <sean.anderson@seco.com>
---

 drivers/phy/xilinx/phy-zynqmp.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/phy/xilinx/phy-zynqmp.c b/drivers/phy/xilinx/phy-zynqmp.c
index b0c2045b886e..2559c6594cea 100644
--- a/drivers/phy/xilinx/phy-zynqmp.c
+++ b/drivers/phy/xilinx/phy-zynqmp.c
@@ -904,7 +904,10 @@ static int xpsgtr_get_ref_clocks(struct xpsgtr_dev *gtr_dev)
 		rate = clk_get_rate(clk);
 
 		for (i = 0 ; i < ARRAY_SIZE(ssc_lookup); i++) {
-			if (rate == ssc_lookup[i].refclk_rate) {
+			/* Allow an error of 100 ppm */
+			unsigned long error = ssc_lookup[i].refclk_rate / 10000;
+
+			if (abs(rate - ssc_lookup[i].refclk_rate) < error) {
 				gtr_dev->refclk_sscs[refclk] = &ssc_lookup[i];
 				break;
 			}
-- 
2.35.1.1320.gc452695387.dirty