[coreboot-gerrit] Change in coreboot[master]: mb/google/brya/var/vell: Add AMP driver setting

[Subrata Banik (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Robert Chen, Nick Vaccaro, Shon Wang.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60331 )

Change subject: mb/google/brya/var/vell: Add AMP driver setting
......................................................................

Patch Set 7:

(2 comments)

Patchset:

PS7:
> I will review in more details tomorrow.
>
> But here is my quick feedback.
>
> I'm recommending to split this CL into atleast 3 CLs as below:
> Add I2C driver for CS35L53 support.

Please consider to check CB:52395 commit msg and while splitting this CL.
> Configure GPIO for vell variant.
> Let Brya Kconfig.name to select this for vell variant and required override dt changes.

File src/drivers/i2c/cs35l53/cs35l53.c:

https://review.coreboot.org/c/coreboot/+/60331/comment/6a44514c_afba9391 
PS7, Line 2:
> one interesting observation that I have post reviewing this Cirrus Logic 35L53 and Cirrus Logic CS42 […]

@Tim/Nick, what do you think ?