arm64: dts: rockchip: use canonical compatible for touchpad/touchscreen
diff mbox

Message ID 20180501224628.GA150745@dtor-ws
State New
Headers show

Commit Message

Dmitry Torokhov May 1, 2018, 10:46 p.m. UTC
"atmel,atmel_mxt_tp" and "atmel,atmel_mxt_ts" are ChromeOS inventions,
let's replace them with canonical compatible string "atmel,maxtouch".

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
 arch/arm64/boot/dts/rockchip/rk3399-gru-kevin.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Brian Norris May 2, 2018, 4:53 p.m. UTC | #1
On Tue, May 1, 2018 at 3:46 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> "atmel,atmel_mxt_tp" and "atmel,atmel_mxt_ts" are ChromeOS inventions,
> let's replace them with canonical compatible string "atmel,maxtouch".
>
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

I guess the other IDs are in the driver just for non-DT usage? Anyway,
seems fine:

Reviewed-by: Brian Norris <briannorris@chromium.org>
Heiko Stuebner May 3, 2018, 12:36 p.m. UTC | #2
Am Mittwoch, 2. Mai 2018, 00:46:28 CEST schrieb Dmitry Torokhov:
> "atmel,atmel_mxt_tp" and "atmel,atmel_mxt_ts" are ChromeOS inventions,
> let's replace them with canonical compatible string "atmel,maxtouch".
> 
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

I've amended the patch subject with a "on gru-kevin" and applied
it for 4.18 with Brian's Reviewed-by


Thanks
Heiko

Patch
diff mbox

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-gru-kevin.dts b/arch/arm64/boot/dts/rockchip/rk3399-gru-kevin.dts
index 191a6bcb17040..82179125bfb7c 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-gru-kevin.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-gru-kevin.dts
@@ -255,7 +255,7 @@  ap_i2c_dig: &i2c2 {
 
 &ap_i2c_tp {
 	trackpad@4a {
-		compatible = "atmel,atmel_mxt_tp";
+		compatible = "atmel,maxtouch";
 		reg = <0x4a>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&trackpad_int_l>;
@@ -271,7 +271,7 @@  ap_i2c_dig: &i2c2 {
 
 &ap_i2c_ts {
 	touchscreen@4b {
-		compatible = "atmel,atmel_mxt_ts";
+		compatible = "atmel,maxtouch";
 		reg = <0x4b>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&touch_int_l>;