Change in coreboot[master]: mb/google/guybrush/nipperkin: update nipperkin config

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Kevin Chiu, Paul Menzel, Eric Peers, Rob Barnes, Felix Held.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57263 )

Change subject: mb/google/guybrush/nipperkin: update nipperkin config
......................................................................

Patch Set 2:

(3 comments)

File src/mainboard/google/guybrush/variants/nipperkin/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/57263/comment/19646705_3a18afbc 
PS2, Line 57: /D2
Nit: No D2.

https://review.coreboot.org/c/coreboot/+/57263/comment/9666c036_79e5bda1 
PS2, Line 76: 	device ref i2c_0 on
            : 		chip drivers/i2c/generic
            : 			register "hid" = ""ELAN0000""
            : 			register "desc" = ""ELAN Touchpad""
            : 			register "irq_gpio" = "ACPI_GPIO_IRQ_EDGE_LOW(GPIO_9)"
            : 			register "wake" = "GEVENT_22"
            : 			register "probed" = "1"
            : 			device i2c 15 on end
            : 		end
            : 	end # I2C0
            : 
            : 	device ref i2c_1 on
            : 		chip drivers/i2c/hid
            : 			register "generic.hid" = ""GDIX0000""
            : 			register "generic.desc" = ""Goodix Touchscreen""
            : 			register "generic.irq_gpio" = "ACPI_GPIO_IRQ_LEVEL_LOW(GPIO_89)"
            : 			register "generic.probed" = "1"
            : 			register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_68)"
            : 			register "generic.enable_delay_ms" = "10"
            : 			register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_121)"
            : 			register "generic.reset_off_delay_ms" = "1"
            : 			register "generic.reset_delay_ms" = "10"
            : 			register "generic.stop_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_120)"
            : 			register "generic.stop_delay_ms" = "170"
            : 			register "generic.stop_off_delay_ms" = "1"
            : 			register "generic.has_power_resource" = "1"
            : 			register "hid_desc_reg_offset" = "0x01"
            : 			device i2c 5d on end
            : 		end
I dont see the build details anywhere. Is it exactly same as guybrush?

https://review.coreboot.org/c/coreboot/+/57263/comment/1079fa1c_d46f5375 
PS2, Line 144: chip drivers/i2c/generic
             : 			register "hid" = ""10EC1019""
             : 			register "desc" = ""Realtek SPK AMP L1""
             : 			register "uid" = "2"
             : 			register "probed" = "1"
             : 			device i2c 2a on end
             : 		end
Either Speaker Amp L1 or Speaker Amp L is sufficient and we dont need both.

In Guybrush Proto0, SAR sensor had I2C address conflict with speaker amplifier L. Hence in Proto 2, speaker amplifier is programmed with new I2C address in L1. That is why I feel you dont need both and just one is sufficient.