From patchwork Sat Nov 30 01:44:37 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13889069 Received: from mail-lf1-f45.google.com (mail-lf1-f45.google.com [209.85.167.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 99F59143871 for ; Sat, 30 Nov 2024 01:45:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.45 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732931130; cv=none; b=lG+0HXah3ktAB6orffEfWVKPv9yVayFX0UJZxlWP4kCjR5WTNk5CbRNXVtTkSMbOcALMaFM2k197o1JLxLTu7WOMDD2XF4iBwfOnPzih6mtxuFNrEuOZxgHEDOVzaBnlLPFSQ6TAcjVbqNjDViiC5BWt3+HfZZNwGAUPJmiCZDM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732931130; c=relaxed/simple; bh=JjPpXCp+YkT/eXiY1YYIwN7gXkPlXjyGgVYw6L4BL40=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=K55N+ydijhXFZo5gIKTMACCdsc5BhhsztjbHhubK9ZyQ0atO1/BfciSNEOU5/gN3iGgYGrfBRYAI6t3ntl0upu45Jla9Xa4hi1DBKzjazZoFG2NIOLSyIZf8IzRh47cdoSJefJQrztCN+KC9tOqoK+4U/f/A3QpFq3rwqk7JxPc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=eA7mt30U; arc=none smtp.client-ip=209.85.167.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="eA7mt30U" Received: by mail-lf1-f45.google.com with SMTP id 2adb3069b0e04-53da6801c6dso2375876e87.0 for ; Fri, 29 Nov 2024 17:45:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1732931127; x=1733535927; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=g4DhC3X/gaoKikUDQE86mzH2vJHzM9VbOVLew/6Upq8=; b=eA7mt30UVzReZzkkBctVJQF/iSNXOWN9OIRWp7d6b8R6lJLkQkFgKMuhPvHRjyPrtN yDJcMLUQs0Ofz+cRdpdPNgvP/n10aQLNfZsPuVoB3A76KeDIECc/lZpQ/38E+ks20bkf mVfGWcR69HV8tmWBdXqX9avQyztPbq2j2NlhxabqdNcjyPibVyVK+3ydos7DbYtcmuib MxLXDdp2oVWteuRxNTgIPA9xdlstgOd7wyGAXPAGe0tEZl9hnKmj60Brn6e5pBWTywXX cWZfWzfHeW+UEVdc2jWoCsNeHmw2z0BtYiNyQVRK4csVNj88lzj3P0g30SfBeCh7FT9t f0Rg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732931127; x=1733535927; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=g4DhC3X/gaoKikUDQE86mzH2vJHzM9VbOVLew/6Upq8=; b=dvukVKKPTRlWiZ2dbkGyyVfqiHR0wwpX07gMtntspyRFIFfXjPsqCMYGjR1HF8rw/4 3QytjpQzkkKNe/B+BudEcBDnjw+rurPM9zlITKdsUlXHw51Gzk9/lZCUlGkH6QIfUeB8 N9gEHN/lMEK9jsse/p/5ly3oO6ZrqnDj3K7ySK1MeTUE7+V1mtq+o8wjiGC5hkZHJ+/x 2FDIjIiacWexm9v6e3tGF0DH2LZB+BwTZ+2UJreCQUs2sMU0snQcmFafEHUUYBhj2aCf aHNON57HwPf+uFe9/lkrLjRNagbvWC53+ecvvWjbOzs4aPJYXtRqZ3pMOnSH+HVBecl3 077Q== X-Forwarded-Encrypted: i=1; AJvYcCUah+LXneXYPEeWMhemNXGDLYJRdwh1v366zHR62FwgDlxWlbjhZKXtru9GqFMh/pWAxWpSVAThPM4wvIke@vger.kernel.org X-Gm-Message-State: AOJu0YwV7KBuYV2fKI0RjZ8LKG9inkkxXWX89aodm2eiN5ku5flyya9x bIBZzW5Zv/DTHXGc8GNx3Om3/raTZmVeOzeNf+QWbIpD0L3sRCnQV8j5epANayg= X-Gm-Gg: ASbGncv07RaqpgVHU6fOkLUkYReoKmefyBWbkjs6UktBaAwzAKYpaqHU2L6MCezymKH Qt2dl/jxGtl5buPgqJUQYW6LiyXH1TBCnM9/LnZFCH7oravp243tICQSCRJzOv0Alr86qm7dmJi i+Yhm3Sv0ro4jDag0RHrBIcEDbQyJGi3jmmBQhbXPofY5dB6mB0MT4L0RczgwPrhhHB65F/u2C7 SF5rm6p3IoJKWueJCodxOAzwjIOU02QbeWlM7Oq+rkrVAmHSga9BYmx8A== X-Google-Smtp-Source: AGHT+IGB+9h29MUOUDhTnmaew4NYiuYWPs0bipcZTTDtdtgzkoVEWLqWDkZP+Ni6xLj44x3hr4T/Cg== X-Received: by 2002:a05:6512:b1c:b0:53d:d19f:1c7e with SMTP id 2adb3069b0e04-53df010ea53mr9076858e87.51.1732931126658; Fri, 29 Nov 2024 17:45:26 -0800 (PST) Received: from umbar.lan ([192.130.178.90]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-53df64a0742sm631946e87.261.2024.11.29.17.45.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Nov 2024 17:45:25 -0800 (PST) From: Dmitry Baryshkov Date: Sat, 30 Nov 2024 03:44:37 +0200 Subject: [PATCH v2 25/31] arm64: dts: qcom: sc8280xp: move board clocks to sc8280xp.dtsi file Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20241130-fix-board-clocks-v2-25-b9a35858657e@linaro.org> References: <20241130-fix-board-clocks-v2-0-b9a35858657e@linaro.org> In-Reply-To: <20241130-fix-board-clocks-v2-0-b9a35858657e@linaro.org> To: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Leo Yan , Joseph Gates , Georgi Djakov , Shawn Guo , Stephan Gerhold , Zac Crosby , =?utf-8?q?Bastian_K=C3=B6cher?= , Andy Gross , Jeremy McNicoll , Rohit Agarwal , Melody Olvera , Bhupesh Sharma , cros-qcom-dts-watchers@chromium.org, Stephen Boyd , Rajendra Nayak , Martin Botka , Jonathan Marek , Vinod Koul , Tengfei Fan , Fenglin Wu , Neil Armstrong , Abel Vesa , Alexandru Marc Serdeliuc , Vladimir Zapolskiy , Sibi Sankar , Bryan O'Donoghue , Jun Nie , James Willcox , Max Chen , Vincent Knecht , Benjamin Li , Krzysztof Kozlowski , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Dmitry Baryshkov X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3869; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=JjPpXCp+YkT/eXiY1YYIwN7gXkPlXjyGgVYw6L4BL40=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBnSm32JloeBcaHMD8trR1qmOsBaxUwdHmP74ZvY kJFzPnM5DqJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZ0pt9gAKCRCLPIo+Aiko 1WTOB/45i3FcA/MSkn4F8pspIU946puZFdxeMttvqaZOPviCYKTQz9KgKvXTNEz83GvSGH4p2WL EDaq7YnwoMZeXJsKw/5cVeklyKHSsIDSY9JkdaS8PeVtJxx6hRqJHpdOwxmC3s9JWr/CxwWRgMH /sUiA1iB3khayI1tBOoaP30W8O/d5HmLHrBsH7ah/dUlxttxaPvSszeQhtRwCIB3BF1l1setmYN EhBfHvhmsXjXJRLQ3rHmXlvVZ5nqPeOVfcimDXyz+2HaY9sdD2X2+EnsSwRDzfZppHrSOq2Tur3 +C/hlPl/mBrvqLDMx8X0fF6fntbYtD+lXKUU5KYs3KbPNMQA X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A SC8280XP, SA8540P and SA8295P have main XO clock definition split between the SoC dtsi file and the board file. This is not optimal, as the clocks are a part of the SoC + PMICs design. Frequencies are common for the whole set of devices using the same SoC. Remove the split and move frequencies to the SoC DTSI file. Suggested-by: Bjorn Andersson Signed-off-by: Dmitry Baryshkov --- arch/arm64/boot/dts/qcom/sa8295p-adp.dts | 4 ---- arch/arm64/boot/dts/qcom/sa8540p-ride.dts | 4 ---- arch/arm64/boot/dts/qcom/sc8280xp-crd.dts | 4 ---- arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 4 ---- arch/arm64/boot/dts/qcom/sc8280xp-microsoft-arcata.dts | 4 ---- arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 1 + 6 files changed, 1 insertion(+), 20 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts index 2fd1dafe63ce7a4f409d19946b9f10ffe324fba3..c536df2acc66276a0bf4e3b2640ee204109867f7 100644 --- a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts +++ b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts @@ -728,10 +728,6 @@ &usb_2_qmpphy1 { status = "okay"; }; -&xo_board_clk { - clock-frequency = <38400000>; -}; - /* PINCTRL */ &pmm8540a_gpios { diff --git a/arch/arm64/boot/dts/qcom/sa8540p-ride.dts b/arch/arm64/boot/dts/qcom/sa8540p-ride.dts index 177b9dad6ff703467ea4d10e0f5a651d11569275..64c35a72a828c128f9dbe9e3537a3f777d21713b 100644 --- a/arch/arm64/boot/dts/qcom/sa8540p-ride.dts +++ b/arch/arm64/boot/dts/qcom/sa8540p-ride.dts @@ -503,10 +503,6 @@ &usb_2_qmpphy0 { status = "okay"; }; -&xo_board_clk { - clock-frequency = <38400000>; -}; - /* PINCTRL */ &tlmm { diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts index 75adaa19d1c3e0c8fcad40c7d0b0afb504331d75..61ae4d0a9e7d10387a8331fbc22d5f88edaa9c23 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts +++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts @@ -868,10 +868,6 @@ &usb_1_qmpphy_out { remote-endpoint = <&pmic_glink_con1_ss>; }; -&xo_board_clk { - clock-frequency = <38400000>; -}; - /* PINCTRL - additions to nodes defined in sc8280xp.dtsi */ &pmc8280_1_gpios { diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts index f3190f408f4b2c7ca9fe368d652d5d66bb3949af..bee1e143178be5971a5916c368a5e8c864927b47 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts +++ b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts @@ -1481,10 +1481,6 @@ &wsamacro { status = "okay"; }; -&xo_board_clk { - clock-frequency = <38400000>; -}; - /* PINCTRL */ &lpass_tlmm { diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-microsoft-arcata.dts b/arch/arm64/boot/dts/qcom/sc8280xp-microsoft-arcata.dts index ae5daeac8fe284bbec86622c10e6831d60a25297..c4a19ecbd894cbdd0521bbf784efd23f46eb07f0 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp-microsoft-arcata.dts +++ b/arch/arm64/boot/dts/qcom/sc8280xp-microsoft-arcata.dts @@ -837,10 +837,6 @@ &wsamacro { status = "okay"; }; -&xo_board_clk { - clock-frequency = <38400000>; -}; - /* PINCTRL */ &lpass_tlmm { diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi index ef06d1ac084d3205a149feff880f457822dca2b4..e6517ad9408d9a49da9c00bcb661f32539fa32e6 100644 --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi @@ -31,6 +31,7 @@ clocks { xo_board_clk: xo-board-clk { compatible = "fixed-clock"; #clock-cells = <0>; + clock-frequency = <38400000>; }; sleep_clk: sleep-clk {