From nobody Thu Apr 16 03:43:57 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C0C8C4332F for ; Wed, 23 Nov 2022 21:17:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235632AbiKWVRX (ORCPT ); Wed, 23 Nov 2022 16:17:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35988 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235288AbiKWVRM (ORCPT ); Wed, 23 Nov 2022 16:17:12 -0500 Received: from mail-qt1-x831.google.com (mail-qt1-x831.google.com [IPv6:2607:f8b0:4864:20::831]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 95745A314A for ; Wed, 23 Nov 2022 13:17:09 -0800 (PST) Received: by mail-qt1-x831.google.com with SMTP id l15so29378qtv.4 for ; Wed, 23 Nov 2022 13:17:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=timesys-com.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=4aqA0Mj3WFRM9SvHIljdHQ8+p96l5z6GhwHZgBiTLAE=; b=HOfGOF/KzrNbTIEs25w7CIh7DweUpRjCaPuDjlosBtlXKTk5QqFEzt3NJlVaE74iA+ jA7UAv8BMJsiduZ2LNB6LjS1SQ3WuyoMibpm7eolcRqsiqLLvyhUKhSNeEF4sUsGESwh n+QZCA9GvgGW7ATurAKm+W2fnWmcLbPdkaWDw+LVsC2p4hEzmgOAnczFW9efTzBhqi9w pGreWtPd65aM7ysQ+zIW0zT8Giv73OPlqvwvs64u+gAwgRbZ59rlJq3oQfikHxObzYjT LRppuKzblg5+Qk6uqK2hM6hItqAFhcw94hgYsYMjILCR13DJUb7ZnTQsRERX5hOSc2Gn xsQQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=4aqA0Mj3WFRM9SvHIljdHQ8+p96l5z6GhwHZgBiTLAE=; b=3roqT5bRFqsZIazKeMPW99Lc3foRM8a8d80QWgD0i/foLVUi/Mol4A28R5O04C+P5I EP0dlz8hEADExZzxPEEgA0zVaKM00H9HhB2p80325n43UAMy6IpZhOuA9wUdHYxZaWxg 8DNndqbd4j8y2npKx0lW3hOyvlb10bmHRUihH/C+R/QzZK2Bo+gncRdBMaWsPYWcASQn 2B9maj2UIqsyWDTHa/8wp6cIOXftNFgqZY0oKlHiiNqGoMmu8qg5v373vfLQ16+ST1PN /sQ0dikBG4Ndd4wH001pETIW72XwWNgN/KKFAWiInDkCDQE77GR6E9pHUA0zso7pbtu0 LG8w== X-Gm-Message-State: ANoB5pmvN6miFQkK38QJu7v72CtTT3ToGK2OaK/wJnLcSPB4DRGGJUDH QubFMPYBW7wRI8X3TNC+5myoTQ== X-Google-Smtp-Source: AA0mqf4OXplR5mPC7T71BzFi9Tan1aVm5pq33+5u3HIsJmIGGOTjUuPwUlTFhECijGg6dxTzh4uCpA== X-Received: by 2002:ac8:4d4f:0:b0:3a6:1dea:8c1c with SMTP id x15-20020ac84d4f000000b003a61dea8c1cmr18403042qtv.157.1669238228695; Wed, 23 Nov 2022 13:17:08 -0800 (PST) Received: from nathan-ideapad.. (d-75-76-18-234.oh.cpe.breezeline.net. [75.76.18.234]) by smtp.gmail.com with ESMTPSA id o22-20020ac872d6000000b00399edda03dfsm10203066qtp.67.2022.11.23.13.17.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 23 Nov 2022 13:17:08 -0800 (PST) From: Nathan Barrett-Morrison Cc: nathan.morrison@timesys.com, greg.malysa@timesys.com, Mark Brown , linux-spi@vger.kernel.org (open list:SPI SUBSYSTEM), linux-kernel@vger.kernel.org (open list) Subject: [PATCH] spi: cadence-quadspi: Add upper limit safety check to baudrate divisor Date: Wed, 23 Nov 2022 16:17:05 -0500 Message-Id: <20221123211705.126900-1-nathan.morrison@timesys.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" While bringing up the cadence-quadspi driver on a customer board, I discovered that the baud divisor calculation can exceed the peripheral's maximum in some circumstances. This will prevent it. Signed-off-by: Nathan Barrett-Morrison --- drivers/spi/spi-cadence-quadspi.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/spi/spi-cadence-quadspi.c b/drivers/spi/spi-cadence-qu= adspi.c index 447230547945..250575fb7b0e 100644 --- a/drivers/spi/spi-cadence-quadspi.c +++ b/drivers/spi/spi-cadence-quadspi.c @@ -1119,6 +1119,10 @@ static void cqspi_config_baudrate_div(struct cqspi_s= t *cqspi) /* Recalculate the baudrate divisor based on QSPI specification. */ div =3D DIV_ROUND_UP(ref_clk_hz, 2 * cqspi->sclk) - 1; =20 + /* Maximum baud divisor */ + if (div > CQSPI_REG_CONFIG_BAUD_MASK) + div =3D CQSPI_REG_CONFIG_BAUD_MASK; + reg =3D readl(reg_base + CQSPI_REG_CONFIG); reg &=3D ~(CQSPI_REG_CONFIG_BAUD_MASK << CQSPI_REG_CONFIG_BAUD_LSB); reg |=3D (div & CQSPI_REG_CONFIG_BAUD_MASK) << CQSPI_REG_CONFIG_BAUD_LSB; --=20 2.30.2