diff mbox series

[v1,2/2] Input: goodix - support gt9147 touchpanel

Message ID 1583144308-3781-3-git-send-email-yannick.fertre@st.com (mailing list archive)
State Accepted
Commit b9a1c116846e9ab9d4872d61604207cd5aef09b4
Headers show
Series goodix: support of gt9147 | expand

Commit Message

Yannick FERTRE March 2, 2020, 10:18 a.m. UTC 
Add support for it by adding compatible and supported chip data
(default settings used).
The chip data on GT9147 is similar to GT912, like
- config data register has 0x8047 address
- config data register max len is 240
- config data checksum has 8-bit

Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
---
 drivers/input/touchscreen/goodix.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Bastien Nocera March 2, 2020, 11 a.m. UTC | #1 
On Mon, 2020-03-02 at 11:18 +0100, Yannick Fertre wrote:
> Add support for it by adding compatible and supported chip data
> (default settings used).
> The chip data on GT9147 is similar to GT912, like
> - config data register has 0x8047 address
> - config data register max len is 240
> - config data checksum has 8-bit
> 
> Signed-off-by: Yannick Fertre <yannick.fertre@st.com>


Both patches:
Reviewed-by: Bastien Nocera <hadess@hadess.net>

But you'll likely need to send another update on top of the "Add
support for Goodix GT917S touch controller" patchset, which makes some
changes to goodix_get_chip_data().
diff mbox series

Patch

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 0403102..8851a35 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -251,6 +251,7 @@  static const struct goodix_chip_data *goodix_get_chip_data(u16 id)
 		return &gt911_chip_data;
 
 	case 912:
+	case 9147:
 	case 967:
 		return &gt967_chip_data;
 
@@ -1054,6 +1055,7 @@  static const struct of_device_id goodix_of_match[] = {
 	{ .compatible = "goodix,gt9271" },
 	{ .compatible = "goodix,gt928" },
 	{ .compatible = "goodix,gt967" },
+	{ .compatible = "goodix,gt9147",},
 	{ }
 };
 MODULE_DEVICE_TABLE(of, goodix_of_match);