diff mbox series

arm64: dts: allwinner: pinephone: Use generic sensor node names

Message ID 20201106032055.51530-1-samuel@sholland.org (mailing list archive)
State New, archived
Headers show
Series arm64: dts: allwinner: pinephone: Use generic sensor node names | expand

Commit Message

Samuel Holland Nov. 6, 2020, 3:20 a.m. UTC
Instead of duplicating part of the compatible string in the node name,
use generic names as recommended by (and listed in) section 2.2.2 of the
Devicetree Specification.

Suggested-by: Maxime Ripard <maxime@cerno.tech>
Signed-off-by: Samuel Holland <samuel@sholland.org>
---
 arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Maxime Ripard Nov. 6, 2020, 11:24 a.m. UTC | #1
On Thu, Nov 05, 2020 at 09:20:55PM -0600, Samuel Holland wrote:
> Instead of duplicating part of the compatible string in the node name,
> use generic names as recommended by (and listed in) section 2.2.2 of the
> Devicetree Specification.
> 
> Suggested-by: Maxime Ripard <maxime@cerno.tech>
> Signed-off-by: Samuel Holland <samuel@sholland.org>

Applied, thanks!
Maxime
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi
index 2da69450eec1..2dfe9bae8c67 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi
@@ -161,7 +161,7 @@  &i2c1 {
 	status = "okay";
 
 	/* Magnetometer */
-	lis3mdl: lis3mdl@1e {
+	lis3mdl: magnetometer@1e {
 		compatible = "st,lis3mdl-magn";
 		reg = <0x1e>;
 		vdd-supply = <&reg_dldo1>;
@@ -169,7 +169,7 @@  lis3mdl: lis3mdl@1e {
 	};
 
 	/* Light/proximity sensor */
-	stk3311@48 {
+	light-sensor@48 {
 		compatible = "sensortek,stk3311";
 		reg = <0x48>;
 		interrupt-parent = <&pio>;
@@ -179,7 +179,7 @@  stk3311@48 {
 	};
 
 	/* Accelerometer/gyroscope */
-	mpu6050@68 {
+	accelerometer@68 {
 		compatible = "invensense,mpu6050";
 		reg = <0x68>;
 		interrupt-parent = <&pio>;