Attention is currently required from: Eran Mitrani, Eric Lai, Jingyuan Liang, Kyoung Il Kim, Paul Menzel.
16 comments:
Commit Message:
Patch Set #2, Line 9: Support WACOM, ELAN, and generic touch sensor devices.
Please write an introduction what that new device it, and what driver is needed for it.
Done
Patch Set #2, Line 10: This driver generates the following ACPI objects:
Please add a blank line between paragraphs.
Done
_DSM
_CRS
Power resource with _STA, _ON, and _OFF
_RST
Please format this as a list.
Done
Patchset:
Separate the ELAN and WACOM support to another patch?
This driver is intended to support these two devices to reduce the needs of introducing all necessary parameters in the devicetree. These two headers are included in its touch.c and it makes sense to stay in single patchset:)
File src/drivers/intel/touch/chip.h:
Mention the datasheet name somewhere in the file?
Done
Patch Set #2, Line 29: Touch Host Controller Mode
THC's 3 protocol modes.
Done
Patch Set #2, Line 30: Switch between Intel THC protocol and Industry standard HID Over SPI protocol
Switch among IPTS, HID-SPI and HID-I2C.
Done
Patch Set #2, Line 31: supporded
supported
Done
THC?
Done
Please add a space after the colon.
Done
Patch Set #2, Line 33: supporded
supported
Done
THC HID I2C is for future support.
Done
/* Touch Host Controller Mode
Switch between Intel THC protocol and Industry standard HID Over SPI protocol.
0x0:Thc IPTS (not supporded at this time)
0x1:Thc HID SPI
0x2 Thc HID I2C; (not supporded at this time)
*/
I’d use […]
Done
Patch Set #2, Line 41: /* GPIO used to enable device. */
Redundant as the variable name says the same?
These parameters follow the GPIO ones in./src/soc/intel/common/block/pcie/rtd3/chip.h:
/* GPIO used to enable device. */
struct acpi_gpio enable_gpio;
/* Delay to be inserted after device is enabled. */
unsigned int enable_delay_ms;
/* Delay to be inserted after device is disabled. */
unsigned int enable_off_delay_ms;
/* GPIO used to take device out of reset or to put it into reset. */
struct acpi_gpio reset_gpio;
/* Delay to be inserted after device is taken out of reset. */
unsigned int reset_delay_ms;
/* Delay to be inserted after device is put into reset. */
unsigned int reset_off_delay_ms;
Other chip.h files also follow the same way.
Patch Set #2, Line 46: enable_off
Why not disable?
same as the above, #41.
File src/drivers/intel/touch/elan.h:
Patch Set #2, Line 9: #define ELAN_RST_SEQ_DLY 50
Is that time? Put the unit into the name?
Done
To view, visit change 81331. To unsubscribe, or for help writing mail filters, visit settings.