[coreboot-gerrit] Change in coreboot[master]: mb/intel/jasperlake_rvp: Camera remove DSDT and enable SSDT

[Sugnan Prabhu S (Code Review)]

Sugnan Prabhu S has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41758 )

Change subject: mb/intel/jasperlake_rvp: Camera remove DSDT and enable SSDT
......................................................................

Patch Set 16:

(3 comments)

https://review.coreboot.org/c/coreboot/+/41758/15/src/mainboard/intel/jasper... 
File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/41758/15/src/mainboard/intel/jasper... 
PS15, Line 374: CAM1
> This is not the correct name for the power resource, it was "PRIC". […]

Done

https://review.coreboot.org/c/coreboot/+/41758/15/src/mainboard/intel/jasper... 
PS15, Line 387: register "chip_name" = ""Camera and Imaging Subsystem"
> Nit: this line probably isn't needed. […]

Done

https://review.coreboot.org/c/coreboot/+/41758/15/src/mainboard/intel/jasper... 
PS15, Line 397: pci 05.0 on end # RP4
> This is not producing any output, exploring on adding a minimal IPU PCI driver.

Done