[coreboot-gerrit] [S] Change in coreboot[main]: mb/google/nissa/var/rull: add touchpad init config of Synaptics&PIXART

[Subrata Banik (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eric Lai, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Rui Zhou.

Subrata Banik has posted comments on this change by Rui Zhou. ( https://review.coreboot.org/c/coreboot/+/84999?usp=email )

Change subject: mb/google/nissa/var/rull: add touchpad init config of Synaptics&PIXART
......................................................................

Patch Set 6:

(1 comment)

File src/mainboard/google/brya/variants/rull/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/84999/comment/54781cb0_a34f7db1?usp... :
PS4, Line 253: 			end
> i found multiple touchpads submitted by coreboot, and it seems that none of them use FW_config to probe; and we use different i2c addresses, so it doesn't matter even if we don't distinguish them.

that is true but it will create ACPI soft device entry. you can assume something like this, if someone is taking this device and booting to OS then they will see many inactive device entries due to redundant ACPI entries