diff mbox series

[v2,2/5] ARM: dts: qcom: asus,nexus7-flo: remove duplicate pinctrl handle in i2c nodes

Message ID 20240711110545.31641-4-rayyan.ansari@linaro.org (mailing list archive)
State Accepted
Commit 6dbec1c39d3ff53e24e434862c7b7da3552b1ffe
Headers show
Series ARM: dts: qcom: adhere to pinctrl dt schema | expand

Commit Message

Rayyan Ansari July 11, 2024, 11:01 a.m. UTC
Remove duplicate handle to i2c pins in the device tree, as they are
already set in qcom-apq8064.dtsi.

Signed-off-by: Rayyan Ansari <rayyan.ansari@linaro.org>
---
v1 -> v2: split from main apq8064 patch 

 arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts | 4 ----
 1 file changed, 4 deletions(-)

Comments

Konrad Dybcio July 11, 2024, 1:49 p.m. UTC | #1
On 11.07.2024 1:01 PM, Rayyan Ansari wrote:
> Remove duplicate handle to i2c pins in the device tree, as they are
> already set in qcom-apq8064.dtsi.
> 
> Signed-off-by: Rayyan Ansari <rayyan.ansari@linaro.org>
> ---

The difference here is that the SoC DTSI also has sleep mode

but I suppose that's only a gain

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
Krzysztof Kozlowski July 11, 2024, 2:04 p.m. UTC | #2
On 11/07/2024 13:01, Rayyan Ansari wrote:
> Remove duplicate handle to i2c pins in the device tree, as they are
> already set in qcom-apq8064.dtsi.
> 
> Signed-off-by: Rayyan Ansari <rayyan.ansari@linaro.org>

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
index d460743fbb94..947183992850 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064-asus-nexus7-flo.dts
@@ -125,8 +125,6 @@  &gsbi1 {
 &gsbi1_i2c {
 	status = "okay";
 	clock-frequency = <200000>;
-	pinctrl-0 = <&i2c1_pins>;
-	pinctrl-names = "default";
 
 	eeprom@52 {
 		compatible = "atmel,24c128";
@@ -148,8 +146,6 @@  &gsbi3 {
 
 &gsbi3_i2c {
 	clock-frequency = <200000>;
-	pinctrl-0 = <&i2c3_pins>;
-	pinctrl-names = "default";
 	status = "okay";
 
 	trackpad@10 {