From patchwork Tue Feb 14 00:57:51 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 13139244 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 E1C26C636CC for ; Tue, 14 Feb 2023 01:00:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229975AbjBNBAe (ORCPT ); Mon, 13 Feb 2023 20:00:34 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57950 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229645AbjBNBAd (ORCPT ); Mon, 13 Feb 2023 20:00:33 -0500 Received: from mail-pg1-x531.google.com (mail-pg1-x531.google.com [IPv6:2607:f8b0:4864:20::531]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 83FB714487 for ; Mon, 13 Feb 2023 17:00:32 -0800 (PST) Received: by mail-pg1-x531.google.com with SMTP id u75so9251786pgc.10 for ; Mon, 13 Feb 2023 17:00:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=jhxllVnbI2AwFWPN2tKYS/gPDBTeZyTnINmF5vnAF9s=; b=jiksO25M1xbJMh+TgDf925qz1OI33yVXPFpT+lYue4N/aK/2EjPChZIbJ4iopZfMH9 qL3CQlp3Q9OGs9elRk+awGd4bmSX4oldimJ+cw6T7pzAirT+A7PkIorWfcbglVMNxmcY x/zEMdVvjkjXDnoH1Sfjv0OboADk5n9MAOyyI= 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=jhxllVnbI2AwFWPN2tKYS/gPDBTeZyTnINmF5vnAF9s=; b=06dZwUIRfmq2hV/uWWIoTDTrR+o96vc8GXP6tPhRtrvMXY5RDPIIwjxlkc69vXa0gA UnbnZtvwLVnYMSsrnu8roPzlnzObwWkZmlzarb/+bhOd2brtw8BARXIp4E2iLDMlPG9O tuZ/QehMZQdqvUmALx5zrlDk6EPKEzuMMpH470R+Z1YoDz9cMrG3eb1irbP/nx9UnCRq Mu4cJbo2nxbkqdM1Upf/jNfScjsu8RgLV40Ia6neImopQHO7/gjBbNH7dznplMN4HkCX 8pQXb3vl+SCnzmd40wOsrPiNPI4QTWLLTN40uB/b7uzcdkiOgVqK/BP+qpcEOToSbzNJ Bqkg== X-Gm-Message-State: AO0yUKU7Gz2xT27Gcj5M232IYF7Y6QyBBApcirCppF1VuBTxOcbiJgIM p9SMB+hbamsvRCTl4iNw+pZ1Xw== X-Google-Smtp-Source: AK7set8iW4DJ4L1yxyeqR9ZgfBeGOdNmdI6N6GwghbcLJ89R3Z2O7NX985VWqNSUx6RrQj83e5E4xg== X-Received: by 2002:a62:17d2:0:b0:5a8:49c8:8533 with SMTP id 201-20020a6217d2000000b005a849c88533mr397121pfx.8.1676336432029; Mon, 13 Feb 2023 17:00:32 -0800 (PST) Received: from tictac2.mtv.corp.google.com ([2620:15c:9d:2:af55:a232:a032:95ff]) by smtp.gmail.com with ESMTPSA id e22-20020aa78256000000b00592626fe48csm8482914pfn.122.2023.02.13.17.00.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Feb 2023 17:00:31 -0800 (PST) From: Douglas Anderson To: Bjorn Andersson Cc: amstan@chromium.org, swboyd@chromium.org, mka@chromium.org, Douglas Anderson , Andy Gross , Konrad Dybcio , Krzysztof Kozlowski , Rob Clark , Rob Herring , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/2] arm64: dts: qcom: sc7180: Fix trogdor qspi pull direction Date: Mon, 13 Feb 2023 16:57:51 -0800 Message-Id: <20230213165743.1.I6f03f86546e6ce9abb1d24fd9ece663c3a5b950c@changeid> X-Mailer: git-send-email 2.39.1.581.gbfd45094c4-goog MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org Though it shouldn't matter very much, we've decided that it's slightly better to park the qspi lines for trogdor with an internal pulldown instead of an internal pullup. There was a footnote that Cr50 (which connects to these lines too) may have pulldowns configured on one of the data lines and we don't want to have fighting pulls. This also means that if the pulls somehow get left powered in S3 (which I'm uncertain about) that they won't be pulling up lines on an unpowered SPI part. Originally the pullup was picked because SPI transfers are active low and thus the high state is somewhat more "idle", but that really isn't that important because the chip select won't be asserted when the bus is idle. The chip select has a nice external pullup on it that's powered by the same power rail as the SPI flash. This shouldn't have any functionality impact w/ reading/writing the SPI since the lines are always push-pull when SPI transfers are actually taking place. Fixes: 7ec3e67307f8 ("arm64: dts: qcom: sc7180-trogdor: add initial trogdor and lazor dt") Signed-off-by: Douglas Anderson --- arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi index 423630c4d02c..de40abcd18db 100644 --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi @@ -1054,7 +1054,7 @@ &qspi_clk { &qspi_data01 { /* High-Z when no transfers; nice to park the lines */ - bias-pull-up; + bias-pull-down; }; &qup_i2c2_default { From patchwork Tue Feb 14 00:57:52 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 13139245 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 54F0CC6379F for ; Tue, 14 Feb 2023 01:00:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229931AbjBNBAf (ORCPT ); Mon, 13 Feb 2023 20:00:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57970 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230022AbjBNBAe (ORCPT ); Mon, 13 Feb 2023 20:00:34 -0500 Received: from mail-pj1-x102e.google.com (mail-pj1-x102e.google.com [IPv6:2607:f8b0:4864:20::102e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 15E02144B7 for ; Mon, 13 Feb 2023 17:00:34 -0800 (PST) Received: by mail-pj1-x102e.google.com with SMTP id nh19-20020a17090b365300b00233ceae8407so6404552pjb.3 for ; Mon, 13 Feb 2023 17:00:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=yWn9i0V4jviqXHKN2YdmFTxHcOb6y0FVwkDe3mDFK6M=; b=m2PnzqgHnJBv2JUn7LigKLSx5KiB21zizYojzPjYtfYTjuDQbwVm8EDB5jrOT7JzjX 66WOjXVgDf+EqBHClQN82HQlJNLxhk5k9mlv4q91tKTu6Y7d0XyKNbK1Pdj/Ssd/PAyX Wk8rQdW0XBTMFaa9EhwneE0mk/s57vTR8VZO0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=yWn9i0V4jviqXHKN2YdmFTxHcOb6y0FVwkDe3mDFK6M=; b=v9tv1FNyNV1AffdF2wvGZv2tdZmxekqJTXauHdrU0i/2DG4q6MXSyFgdLcLrZPQTji 2SYkhrSsvamcY6yxmj9SGhFq50RP5p00Md/xF+Zxc3X6jOyLOLzT9f6fvcRqb6+osns1 E8/eqDe4lgEVdOh2DIgSfYQz37EvgE9Lj7q4uq+vDx3SlwtQoR9Ue5ObDwFR8h0r2Rjc CbbeQyp/cb/0lOHJ0VT28/Hb3vtHhnue22+oxTVv7qMEvC0y40eEzbxU3aP1ddOkvJcw ZIlOQia+/1U1GDENPcQQbFCqx01iEz3sNECLMSbwW0xXiE0r30WFX6gX5Djj55HJz9KI mH9Q== X-Gm-Message-State: AO0yUKUCGKcsUIn8+eKOy065H/K4GJLYz1sS9jLqiTnoTbfw+rGAr8Ia J2e8ke6dTPmozABhrULyj//zWg== X-Google-Smtp-Source: AK7set8sGpBqSf//1hjHWq+NO6lviBTQytuDGRQZdpS7yQO9CR0Ya3vQMWzL0MqDJBCSVmvDmHO0uQ== X-Received: by 2002:a05:6a20:1453:b0:c0:c938:8c31 with SMTP id a19-20020a056a20145300b000c0c9388c31mr146109pzi.14.1676336433546; Mon, 13 Feb 2023 17:00:33 -0800 (PST) Received: from tictac2.mtv.corp.google.com ([2620:15c:9d:2:af55:a232:a032:95ff]) by smtp.gmail.com with ESMTPSA id e22-20020aa78256000000b00592626fe48csm8482914pfn.122.2023.02.13.17.00.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Feb 2023 17:00:33 -0800 (PST) From: Douglas Anderson To: Bjorn Andersson Cc: amstan@chromium.org, swboyd@chromium.org, mka@chromium.org, Douglas Anderson , Andy Gross , Konrad Dybcio , Krzysztof Kozlowski , Rob Herring , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/2] arm64: dts: qcom: sc7280: Fix herobrine qspi pull direction Date: Mon, 13 Feb 2023 16:57:52 -0800 Message-Id: <20230213165743.2.I06f9e461a85fcd5d6fb1e977aa253f6523096b6f@changeid> X-Mailer: git-send-email 2.39.1.581.gbfd45094c4-goog In-Reply-To: <20230213165743.1.I6f03f86546e6ce9abb1d24fd9ece663c3a5b950c@changeid> References: <20230213165743.1.I6f03f86546e6ce9abb1d24fd9ece663c3a5b950c@changeid> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org Though it shouldn't matter very much, we've decided that it's slightly better to park the qspi lines for herobrine with an internal pulldown instead of an internal pullup. There is an external pulldown on one of the data lines on the board and we don't want to have fighting pulls. This also means that if the pulls somehow get left powered in S3 (which I'm uncertain about) that they won't be pulling up lines on an unpowered SPI part. Originally the pullup was picked because SPI transfers are active low and thus the high state is somewhat more "idle", but that really isn't that important because the chip select won't be asserted when the bus is idle. The chip select has a nice external pullup on it that's powered by the same power rail as the SPI flash. This shouldn't have any functionality impact w/ reading/writing the SPI since the lines are always push-pull when SPI transfers are actually taking place. Fixes: 116f7cc43d28 ("arm64: dts: qcom: sc7280: Add herobrine-r1") Signed-off-by: Douglas Anderson --- arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi index b6137816f2f3..7d787b12c10f 100644 --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi @@ -703,7 +703,7 @@ &qspi_clk { &qspi_data01 { /* High-Z when no transfers; nice to park the lines */ - bias-pull-up; + bias-pull-down; drive-strength = <8>; };