Re: [PATCH v4 2/2] HID: i2c-hid: elan: Add ili9882t timing

From
Doug Anderson <dianders@chromium.org>
Date
2023-06-08 13:39:52
Also in
linux-devicetree, lkml
Reply to
Cong Yang, 2023-06-08
Hi,

On Thu, Jun 8, 2023 at 6:02 AM Cong Yang
[off-list ref] wrote:
quoted hunk ↗ jump to hunk

+static const struct elan_i2c_hid_chip_data ilitek_ili9882t_chip_data = {
+       .post_power_delay_ms = 1,
+       .post_gpio_reset_on_delay_ms = 200,
+       .post_gpio_reset_off_delay_ms = 65,
        .hid_descriptor_address = 0x0001,
+       /* this touchscreen is tightly integrated with the panel and assumes
+        * that the relevant power rails (other than the IO rail) have already
+        * been turned on by the panel driver because we're a panel follower.
+        */
I don't think the above is the right block comment style for this
file. The first "/*" should be on a line on its own.

+       .main_supply_name = "null",
This should actually be NULL without the quotes. ...then you should
change the code not to try to grab the regulator in that case.
Thread (8 messages) 8 messages, 3 authors, last reply 2023-06-09
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help