Varshit B Pandya has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
Add support as per the schmatics Add 2 Ports and 2 Endpoints Add support for OTVI5676
Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 11 files changed, 611 insertions(+), 16 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/1
diff --git a/src/mainboard/intel/jasperlake_rvp/Kconfig b/src/mainboard/intel/jasperlake_rvp/Kconfig index 97b038e..edf1bea 100644 --- a/src/mainboard/intel/jasperlake_rvp/Kconfig +++ b/src/mainboard/intel/jasperlake_rvp/Kconfig @@ -50,4 +50,10 @@ config UART_FOR_CONSOLE int default 2 + +config VARIANT_HAS_CAMERA_ACPI + bool + default n + help + Select this option to enable camera ACPI support on the variant endif diff --git a/src/mainboard/intel/jasperlake_rvp/Kconfig.name b/src/mainboard/intel/jasperlake_rvp/Kconfig.name index 1a56f05..d0fa41e 100644 --- a/src/mainboard/intel/jasperlake_rvp/Kconfig.name +++ b/src/mainboard/intel/jasperlake_rvp/Kconfig.name @@ -1,5 +1,6 @@ config BOARD_INTEL_JASPERLAKE_RVP bool "Jasperlake DDR4/LPDDR4 RVP" + select VARIANT_HAS_CAMERA_ACPI
config BOARD_INTEL_JASPERLAKE_RVP_EXT_EC bool "Jasperlake DDR4/LPDDR4 RVP with Chrome EC" diff --git a/src/mainboard/intel/jasperlake_rvp/dsdt.asl b/src/mainboard/intel/jasperlake_rvp/dsdt.asl index c996717..81677df 100644 --- a/src/mainboard/intel/jasperlake_rvp/dsdt.asl +++ b/src/mainboard/intel/jasperlake_rvp/dsdt.asl @@ -41,13 +41,15 @@ } }
-#if CONFIG(CHROMEOS) - /* Chrome OS specific */ - #include <vendorcode/google/chromeos/acpi/chromeos.asl> -#endif
-#if CONFIG(EC_GOOGLE_CHROMEEC) - /* Chrome OS Embedded Controller */ + + #if CONFIG(CHROMEOS) + /* Chrome OS specific */ + #include <vendorcode/google/chromeos/acpi/chromeos.asl> + #endif + + #if CONFIG(EC_GOOGLE_CHROMEEC) + /* Chrome OS Embedded Controller */ Scope (_SB.PCI0.LPCB) { /* ACPI code for EC SuperIO functions */ @@ -55,7 +57,12 @@ /* ACPI code for EC functions */ #include <ec/google/chromeec/acpi/ec.asl> } -#endif + #endif + + #if CONFIG(VARIANT_HAS_CAMERA_ACPI) + /* Camera */ + #include <variant/acpi/camera.asl> + #endif
#include <southbridge/intel/common/acpi/sleepstates.asl>
diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl new file mode 100644 index 0000000..f560d4a --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl @@ -0,0 +1,168 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2020 Intel Corporation. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +Scope (_SB.PCI0.I2C4) +{ + PowerResource (RCPR, 0x00, 0x0000) + { + Name (STA, Zero) + Method (_ON, 0, Serialized) // _ON_: Power On + { + If ((STA == Zero)) + { + // Enable CLK0 + MCCT(0,1,1) // Clock 0, enable, 19.2MHz + // Pull PWREN(GPIO D5) high + STXS(GPP_D5) + Sleep(5) + // Pull RST(GPIO B14) low + CTXS(GPP_B14) + Sleep(5) + // Pull RST high + STXS(GPP_B14) + Sleep(5) + Store(1,STA) + } + } + + Method (_OFF, 0, Serialized) // _OFF: Power Off + { + If ((STA == One)) + { + // Pull RST low + CTXS(GPP_B14) + // Pull PWREN low + CTXS(GPP_D5) + // Disable CLK0 + MCCT(0,0,1) // Clock 0, disable, 19.2MHz + // STA = Zero + Store(0,STA) + } + } + + Method (_STA, 0, NotSerialized) // _STA: Status + { + Return (STA) + } + } + + Device (CAM0) + { + Name (_HID, "OVTI5675") // _HID: Hardware ID + Name (_UID, Zero) // _UID: Unique ID + Name (_DDN, "Ov 5675 Camera") // _DDN: DOS Device Name + Method (_STA, 0, NotSerialized) // _STA: Status + { + Return (0x0F) + } + + Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings + { + I2cSerialBus (0x0036, ControllerInitiated, 0x00061A80, + AddressingMode7Bit, "\_SB.PCI0.I2C4", + 0x00, ResourceConsumer, , + ) + }) + + Name (_PR0, Package (0x01) // _PR0: Power Resources for D0 + { + RCPR + }) + + Name (_PR3, Package (0x01) // _PR3: Power Resources for D3hot + { + RCPR + }) + + Name (_DSD, Package (0x04) // _DSD: Device-Specific Data + { + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "port0", + "PRT0" + } + }, + + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "clock-frequency", + 0x0124F800 + }, + } + }) + + Name (PRT0, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + Zero + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP00" + } + } + }) + + Name (EP00, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x03) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "link-frequencies", + Package (0x01) + { + 0x325AA000 + } + }, + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + IPU0, + Zero, + Zero + } + } + } + }) + } + +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl new file mode 100644 index 0000000..9187ed6 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl @@ -0,0 +1,166 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2020 Intel Corporation. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +Scope (_SB.PCI0.I2C5) +{ + PowerResource (FCPR, 0x00, 0x0000) + { + Name (STA, Zero) + Method (_ON, 0, Serialized) // _ON_: Power On + { + If ((STA == Zero)) + { + // Enable CLK1 + MCCT(1,1,1) // Clock 1, enable, 19.2MHz + // Pull PWREN(GPIO R6) high + STXS(GPP_D4) + Sleep(5) + // Pull RST(GPIO H12) low + CTXS(GPP_C19) + Sleep(5) + // Pull RST high + STXS(GPP_C19) + Sleep(5) + Store(1,STA) + } + } + + Method (_OFF, 0, Serialized) // _OFF: Power Off + { + If ((STA == One)) + { + // Pull RST low + CTXS(GPP_C19) + // Pull PWREN low + CTXS(GPP_D4) + // Disable CLK0 + MCCT(1,0,1) // Clock 1, disable, 19.2MHz + Store(0,STA) + } + } + + Method (_STA, 0, NotSerialized) // _STA: Status + { + Return (STA) + } + } + + Device (CAM1) + { + Name (_HID, "OVTI5675") // _HID: Hardware ID + Name (_UID, Zero) // _UID: Unique ID + Name (_DDN, "Ov 5675 Camera") // _DDN: DOS Device Name + Method (_STA, 0, NotSerialized) // _STA: Status + { + Return (0x0F) + } + + Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings + { + I2cSerialBus (0x0036, ControllerInitiated, 0x00061A80, + AddressingMode7Bit, "\_SB.PCI0.I2C5", + 0x00, ResourceConsumer, , + ) + }) + + Name (_PR0, Package (0x01) // _PR0: Power Resources for D0 + { + FCPR + }) + + Name (_PR3, Package (0x01) // _PR3: Power Resources for D3hot + { + FCPR + }) + + Name (_DSD, Package (0x04) // _DSD: Device-Specific Data + { + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "port0", + "PRT0" + } + }, + + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "clock-frequency", + 0x0124F800 + } + } + }) + + Name (PRT0, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + Zero + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP00" + } + } + }) + + Name (EP00, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x03) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "link-frequencies", + Package (0x01) + { + 0x325AA000 + } + }, + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + IPU0, + One, + Zero + } + } + } + }) + } +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl new file mode 100644 index 0000000..542a274 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl @@ -0,0 +1,19 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2020 Intel Corporation. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include "ipu_mainboard.asl" +#include "ipu_endpoints.asl" +#include "cam0.asl" +#include "cam1.asl" diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl new file mode 100644 index 0000000..f684c6b --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl @@ -0,0 +1,96 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2020 Intel Corporation. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +cope (_SB.PCI0.IPU0) +{ + Name (EP00, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x04) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "clock-lanes", + Zero + }, + + Package (0x02) + { + "data-lanes", + Package (0x02) + { + One, + 0x02, + } + }, + + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + ^I2C4.CAM0, + Zero, + Zero + } + } + } + }) + Name (EP10, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x04) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "clock-lanes", + Zero + }, + + Package (0x02) + { + "data-lanes", + Package (0x02) + { + One, + 0x02, + } + }, + + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + ^I2C5.CAM1, + Zero, + Zero + } + } + } + }) +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl new file mode 100644 index 0000000..64bfc71 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl @@ -0,0 +1,91 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2020 Intel Corporation. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +Scope (_SB.PCI0) +{ + Device (IPU0) + { + Name (_ADR, 0x00050000) // _ADR: Address + Name (_DDN, "Camera and Imaging Subsystem") // _DDN: DOS Device Name + } +} + +Scope (_SB.PCI0.IPU0) +{ + Name (_DSD, Package (0x02) // _DSD: Device-Specific Data + { + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x02) + { + Package (0x02) + { + "port0", + "PRT0" + }, + + Package (0x02) + { + "port1", + "PRT1" + } + } + }) + + Name (PRT0, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + Zero + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP00" + } + } + }) + + Name (PRT1, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + 2 + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP10" + } + } + }) +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb index 9c40f66..946b4be 100644 --- a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb @@ -73,7 +73,7 @@ [PchSerialIoIndexI2C1] = PchSerialIoPci, [PchSerialIoIndexI2C2] = PchSerialIoPci, [PchSerialIoIndexI2C3] = PchSerialIoPci, - [PchSerialIoIndexI2C4] = PchSerialIoDisabled, + [PchSerialIoIndexI2C4] = PchSerialIoPci, [PchSerialIoIndexI2C5] = PchSerialIoPci, }"
@@ -116,13 +116,20 @@ .speed_mhz = 1, .early_init = 1, }, + .i2c[4] = { + .speed = I2C_SPEED_FAST, + }, + .i2c[5] = { + .speed = I2C_SPEED_FAST, + }, }"
device domain 0 on device pci 00.0 on end # Host Bridge device pci 02.0 on end # Integrated Graphics Device - device pci 04.0 off end # SA Thermal device - device pci 12.0 off end # Thermal Subsystem + device pci 04.0 off end # SA Thermal device + device pci 05.0 on end #IPU + device pci 12.0 off end # Thermal Subsystem device pci 12.5 off end # UFS SCS device pci 12.6 off end # GSPI #2 device pci 14.0 on @@ -229,9 +236,9 @@ device i2c 2c on end end end # I2C 0 - device pci 15.1 on end # I2C #1 - device pci 15.2 on end # I2C #2 - device pci 15.3 on end # I2C #3 + device pci 15.1 on end # I2C #1 + device pci 15.2 on end # I2C #2 + device pci 15.3 on end # I2C #3 device pci 16.0 on end # Management Engine Interface 1 device pci 16.1 off end # Management Engine Interface 2 device pci 16.2 off end # Management Engine IDE-R @@ -240,7 +247,7 @@ device pci 16.5 off end # Management Engine Interface 4 device pci 17.0 off end # SATA device pci 19.0 on end # I2C #4 - device pci 19.1 off end # I2C #5 + device pci 19.1 on end # I2C #5 device pci 19.2 on end # UART #2 device pci 1a.0 on end # eMMC device pci 1c.0 off end # PCI Express Port 1 @@ -262,7 +269,7 @@ device spi 0 on end end end # GSPI #1 - device pci 1f.0 on end # eSPI Interface + device pci 1f.0 on end # eSPI Interface device pci 1f.1 on end # P2SB device pci 1f.2 on end # Power Management Controller device pci 1f.3 on end # Intel HDA diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c index 91292b8..850a9b4 100644 --- a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c @@ -18,7 +18,26 @@
/* Pad configuration in ramstage*/ static const struct pad_config gpio_table[] = { - /* ToDo: Fill gpio configuration */ + /* B14 : CAM1_RST_N */ + PAD_CFG_GPO(GPP_B14, 0, PLTRST), + + /* D4 : CAM2_PWREN */ + PAD_CFG_GPO(GPP_D4, 0, PLTRST), + /* D5 : CRD_CAM_PWREN */ + PAD_CFG_GPO(GPP_D5, 0, PLTRST), + /* D12 : GPPC_D_12_I2C4B_SDA_CRD1 */ + PAD_CFG_NF(GPP_D12, NONE, PLTRST, NF3), + /* D13 : GPPC_D_13_I2C4B_SCL_CRD1 */ + PAD_CFG_NF(GPP_D13, NONE, PLTRST, NF3), + /* D22 : GPPC_D_22_I2C5_SDA_CRD2 */ + PAD_CFG_NF(GPP_D22, NONE, PLTRST, NF1), + /* D23 : GPPC_D_23_I2C5_SCL_CRD2 */ + PAD_CFG_NF(GPP_D23, NONE, PLTRST, NF1), + + /* E0 : IMGCLKOUT_0 */ + PAD_CFG_NF(GPP_E0, NONE, PLTRST, NF2), + /* E2 : IMGCLKOUT_1 */ + PAD_CFG_NF(GPP_E2, NONE, PLTRST, NF1), };
/* Early pad configuration in bootblock */ diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl new file mode 100644 index 0000000..0cfdd17 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl @@ -0,0 +1,15 @@ +/*This file is part of the coreboot project. + * + * Copyright 2020 Intel Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <baseboard/acpi/camera.asl>
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 6:
This change is ready for review.
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#7).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 560 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/7
Maulik V Vaghela has uploaded a new patch set (#9) to the change originally created by Varshit B Pandya. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 562 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/9
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#10).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 624 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/10
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#11).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 633 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/11
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 11:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39669/11/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl:
https://review.coreboot.org/c/coreboot/+/39669/11/src/mainboard/intel/jasper... PS11, Line 155: trailing whitespace
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#12).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 633 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/12
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#13).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 634 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/13
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Aamir Bohra, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#14).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 634 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/14
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 14: Code-Review+1
Ronak Kanabar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 14: Code-Review+1
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#15).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5676
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 637 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/15
Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 15:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG@9 PS15, Line 9: 1. Add support as per the schematics : 2. Add 2 Ports and 2 Endpoints nit: Add some more information here, like below World facing sensor on I2C5 using 2 data lanes with VCM support User facing sensor on I2C4 using 2 data lanes
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG@11 PS15, Line 11: OTVI5676 OVTI5675?
https://review.coreboot.org/c/coreboot/+/39669/15/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl:
https://review.coreboot.org/c/coreboot/+/39669/15/src/mainboard/intel/jasper... PS15, Line 49: Device (CAM0) The same sensor in CAM1 has a VCM device? shouldn't this one also contain an entry for VCM?
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#16).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5675 and OVTI2740
WFC is on I2C5 with VCM support and using 2 data-lanes and UFC is on I2C4 using 2 data-lanes
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Kconfig.name M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 12 files changed, 638 insertions(+), 15 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/16
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 16:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG@9 PS15, Line 9: 1. Add support as per the schematics : 2. Add 2 Ports and 2 Endpoints
nit: Add some more information here, like below […]
Updated
https://review.coreboot.org/c/coreboot/+/39669/15//COMMIT_MSG@11 PS15, Line 11: OTVI5676
OVTI5675?
Updated.
https://review.coreboot.org/c/coreboot/+/39669/15/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl:
https://review.coreboot.org/c/coreboot/+/39669/15/src/mainboard/intel/jasper... PS15, Line 49: Device (CAM0)
The same sensor in CAM1 has a VCM device? shouldn't this one also contain an entry for VCM?
Updated the correct sensor details
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#17).
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
1. Add support as per the schematics 2. Add 2 Ports and 2 Endpoints 3. Add support for OTVI5675 and OVTI2740
WFC is on I2C5 with VCM support and using 2 data-lanes and UFC is on I2C4 using 2 data-lanes
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam0.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 11 files changed, 637 insertions(+), 15 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/17
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP ......................................................................
Patch Set 17:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG@7 PS17, Line 7: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP nit: remove Jasper Lake RVP
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG@9 PS17, Line 9: 1. Add support as per the schematics Configure GPIOs as per schematics
https://review.coreboot.org/c/coreboot/+/39669/17/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/Kconfig:
https://review.coreboot.org/c/coreboot/+/39669/17/src/mainboard/intel/jasper... PS17, Line 70: config VARIANT_HAS_CAMERA_ACPI Since its by default y, shall we remove it and add ACPI file by default?
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#18).
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 10 files changed, 374 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/18
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#19).
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
BUG=None BRANCH=None TEST=Build and Boot jslrvp board and able to capture image using world facing camera.
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 10 files changed, 379 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/19
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#20).
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
BUG=None BRANCH=None TEST=Build and Boot jslrvp board and able to capture image using world facing camera.
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Kconfig M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 10 files changed, 371 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/20
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 20:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG@7 PS17, Line 7: mb/intel/jasperlake_rvp: Add camera support Jasper Lake RVP
nit: remove Jasper Lake RVP
Updated.
https://review.coreboot.org/c/coreboot/+/39669/17//COMMIT_MSG@9 PS17, Line 9: 1. Add support as per the schematics
Configure GPIOs as per schematics
Updated.
https://review.coreboot.org/c/coreboot/+/39669/17/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/Kconfig:
https://review.coreboot.org/c/coreboot/+/39669/17/src/mainboard/intel/jasper... PS17, Line 70: config VARIANT_HAS_CAMERA_ACPI
Since its by default y, shall we remove it and add ACPI file by default?
Updated.
Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 20: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/39669/20/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/Kconfig:
https://review.coreboot.org/c/coreboot/+/39669/20/src/mainboard/intel/jasper... PS20, Line 69: unnecessary change
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#21).
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
BUG=None BRANCH=None TEST=Build and Boot jslrvp board and able to capture image using world facing camera.
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 9 files changed, 370 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/21
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 21:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39669/20/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/Kconfig:
https://review.coreboot.org/c/coreboot/+/39669/20/src/mainboard/intel/jasper... PS20, Line 69:
unnecessary change
Updated.
Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 21: Code-Review+2
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 21: Code-Review+2
Ronak Kanabar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 21: Code-Review+2
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 21:
(5 comments)
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: /* This file is part of the coreboot project. */ please remove
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */ please remove
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: /* This file is part of the coreboot project. */ please remove
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */ please remove
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */ please remove
Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Sooraj Govindan, Ronak Kanabar, Aamir Bohra,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39669
to look at the new patch set (#22).
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
BUG=None BRANCH=None TEST=Build and Boot jslrvp board and able to capture image using world facing camera.
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com --- M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 9 files changed, 365 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/39669/22
Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 22:
(5 comments)
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: /* This file is part of the coreboot project. */
please remove
Done
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */
please remove
Done
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: /* This file is part of the coreboot project. */
please remove
Done
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */
please remove
Done
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... File src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl:
https://review.coreboot.org/c/coreboot/+/39669/21/src/mainboard/intel/jasper... PS21, Line 2: * This file is part of the coreboot project. */
please remove
Done
Ronak Kanabar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 22: Code-Review+2
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 22: Code-Review+2
Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 22: Code-Review+2
Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
mb/intel/jasperlake_rvp: Add world facing camera support
1. Configure GPIOs as per schematics 2. Add 1 Ports and 1 Endpoints 3. Add support for OTVI5675
WFC is on I2C5 with VCM support and using 2 data-lanes
BUG=None BRANCH=None TEST=Build and Boot jslrvp board and able to capture image using world facing camera.
Change-Id: I07ae9e3473c16bde8eb1597460e70cc478357b98 Signed-off-by: Pandya, Varshit B varshit.b.pandya@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/39669 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Ronak Kanabar ronak.kanabar@intel.com Reviewed-by: Maulik V Vaghela maulik.v.vaghela@intel.com Reviewed-by: Aamir Bohra aamir.bohra@intel.com --- M src/mainboard/intel/jasperlake_rvp/Makefile.inc M src/mainboard/intel/jasperlake_rvp/dsdt.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl A src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb M src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c A src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl 9 files changed, 365 insertions(+), 14 deletions(-)
Approvals: build bot (Jenkins): Verified Aamir Bohra: Looks good to me, approved Maulik V Vaghela: Looks good to me, approved Ronak Kanabar: Looks good to me, approved
diff --git a/src/mainboard/intel/jasperlake_rvp/Makefile.inc b/src/mainboard/intel/jasperlake_rvp/Makefile.inc index 20b3be4..c1fde49 100644 --- a/src/mainboard/intel/jasperlake_rvp/Makefile.inc +++ b/src/mainboard/intel/jasperlake_rvp/Makefile.inc @@ -19,6 +19,7 @@
subdirs-y += ../common subdirs-y += variants/baseboard +CPPFLAGS_common += -I$(src)/mainboard/$(MAINBOARDDIR)/variants/$(VARIANT_DIR)/include CPPFLAGS_common += -I$(src)/mainboard/$(MAINBOARDDIR)/variants/baseboard/include
subdirs-y += variants/$(VARIANT_DIR) diff --git a/src/mainboard/intel/jasperlake_rvp/dsdt.asl b/src/mainboard/intel/jasperlake_rvp/dsdt.asl index 3e45cca..f47424f 100644 --- a/src/mainboard/intel/jasperlake_rvp/dsdt.asl +++ b/src/mainboard/intel/jasperlake_rvp/dsdt.asl @@ -36,15 +36,18 @@
#if CONFIG(EC_GOOGLE_CHROMEEC) /* Chrome OS Embedded Controller */ - Scope (_SB.PCI0.LPCB) - { - /* ACPI code for EC SuperIO functions */ - #include <ec/google/chromeec/acpi/superio.asl> - /* ACPI code for EC functions */ - #include <ec/google/chromeec/acpi/ec.asl> - } + Scope (_SB.PCI0.LPCB) + { + /* ACPI code for EC SuperIO functions */ + #include <ec/google/chromeec/acpi/superio.asl> + /* ACPI code for EC functions */ + #include <ec/google/chromeec/acpi/ec.asl> + } #endif
+ /* Camera */ + #include <variant/acpi/camera.asl> + #include <southbridge/intel/common/acpi/sleepstates.asl>
/* Mainboard specific */ diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl new file mode 100644 index 0000000..1413998 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/cam1.asl @@ -0,0 +1,226 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +Scope (_SB.PCI0.I2C5) +{ + PowerResource (FCPR, 0x00, 0x0000) + { + Name (STA, Zero) + Method (_ON, 0, Serialized) /* _ON_: Power On */ + { + If ((STA == Zero)) + { + /* Enable CLK1 */ + MCON(1, 1) // Clock 1, 19.2MHz + /* Pull PWREN(GPIO R6) high */ + STXS(GPP_D4) + Sleep(5) /* 5 us */ + /* Pull RST(GPIO H12) low */ + CTXS(GPP_C19) + Sleep(5) /* 5 us */ + /* Pull RST high */ + STXS(GPP_C19) + Sleep(5) /* 5 us */ + STA = 1 + } + } + + Method (_OFF, 0, Serialized) /* _OFF: Power Off */ + { + If ((STA == One)) + { + /* Pull RST low */ + CTXS(GPP_C19) + /* Pull PWREN low */ + CTXS(GPP_D4) + /* Disable CLK0 */ + MCOF(1) /* Clock 1 */ + STA = 0 + } + } + + Method (_STA, 0, NotSerialized) /* _STA: Status */ + { + Return (STA) + } + } + + Device (CAM1) + { + Name (_HID, "OVTI5675") /* _HID: Hardware ID */ + + Name (_UID, Zero) /* _UID: Unique ID */ + + Name (_DDN, "Ov 5675 Camera") /* _DDN: DOS Device Name */ + + Method (_STA, 0, NotSerialized) /* _STA: Status */ + { + Return (0x0F) + } + + Name (_CRS, ResourceTemplate () /* _CRS: Current Resource Settings */ + { + I2cSerialBus (0x0036, ControllerInitiated, 0x00061A80, + AddressingMode7Bit, "\_SB.PCI0.I2C5", + 0x00, ResourceConsumer, , + ) + }) + + Name (_PR0, Package (0x01) /* _PR0: Power Resources for D0 */ + { + FCPR + }) + + Name (_PR3, Package (0x01) /* _PR3: Power Resources for D3hot */ + { + FCPR + }) + + Name (_DSD, Package (0x04) /* _DSD: Device-Specific Data */ + { + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "port0", + "PRT0" + } + }, + + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x02) + { + Package (0x02) + { + "clock-frequency", + 0x0124F800 + }, + + Package (0x02) + { + "lens-focus", + Package (0x01) + { + VCM0 + } + } + } + }) + + Name (PRT0, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + Zero + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP00" + } + } + }) + + Name (EP00, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x04) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "data-lanes", + Package (0x02) + { + One, + 0x02 + } + }, + + Package (0x02) + { + "link-frequencies", + Package (0x01) + { + 0x1AD27480 + } + }, + + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + IPU0, + One, + Zero + } + } + } + }) + } + + Device (VCM0) + { + Name (_HID, "PRP0001") /* _HID: Hadware ID */ + + Name (_UID, 0x03) /* _UID: Unique ID */ + + Name (_DDN, "DW9714 VCM") /* _DDN: DOS Device Name */ + + Method (_STA, 0, NotSerialized) /* _STA: Status */ + { + Return (0x0F) + } + + Name (_CRS, ResourceTemplate () /* _CRS: Current Resource Setting */ + { + I2cSerialBusV2 (0x000C, ControllerInitiated, 0x00061A80, + AddressingMode7Bit, "\_SB.PCI0.I2C5", + 0x00, ResourceConsumer, , Exclusive, + ) + }) + + Name (_DEP, Package (0x01) /* _DEP: Dependencies */ + { + CAM1 + }) + + Name (_PR0, Package (0x01) /* _PR0: Power Resources for D0 */ + { + FCPR + }) + + Name (_PR3, Package (0x01) /* _PR3: Power Resources for D3Hot */ + { + FCPR + }) + + Name (_DSD, Package (0x02) /* _DSD: Device-Specific Data */ + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), /* Device Properties for _DSD */ + Package(0x01) + { + Package (0x02) + { + "compatible", + "dongwoon,dw9714" + } + } + }) + } +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl new file mode 100644 index 0000000..48650c9 --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/camera.asl @@ -0,0 +1,5 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#include "ipu_mainboard.asl" +#include "ipu_endpoints.asl" +#include "cam1.asl" diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl new file mode 100644 index 0000000..cebf15e --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_endpoints.asl @@ -0,0 +1,44 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +Scope (_SB.PCI0.IPU0) +{ + Name (EP10, Package (0x02) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x04) + { + Package (0x02) + { + "endpoint", + Zero + }, + + Package (0x02) + { + "clock-lanes", + Zero + }, + + Package (0x02) + { + "data-lanes", + Package (0x02) + { + One, + 0x02, + } + }, + + Package (0x02) + { + "remote-endpoint", + Package (0x03) + { + ^I2C5.CAM1, + Zero, + Zero + } + } + } + }) +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl new file mode 100644 index 0000000..013b26f --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/baseboard/include/baseboard/acpi/ipu_mainboard.asl @@ -0,0 +1,50 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +Scope (_SB.PCI0) +{ + Device (IPU0) + { + Name (_ADR, 0x00050000) // _ADR: Address + + Name (_DDN, "Camera and Imaging Subsystem") // _DDN: DOS Device Name + } +} + +Scope (_SB.PCI0.IPU0) +{ + Name (_DSD, Package (0x02) // _DSD: Device-Specific Data + { + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "port1", + "PRT1" + } + } + }) + + Name (PRT1, Package (0x04) + { + ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), + Package (0x01) + { + Package (0x02) + { + "port", + 2 + } + }, + + ToUUID ("dbb8e3e6-5886-4ba6-8795-1319f52a966b"), + Package (0x01) + { + Package (0x02) + { + "endpoint0", + "EP10" + } + } + }) +} diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb index 1e88c7a..14ca4a5 100644 --- a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb @@ -137,7 +137,7 @@ .speed_mhz = 1, .early_init = 1, }, - .i2c[0] = { + .i2c[0] = { .speed = I2C_SPEED_FAST, .speed_config[0] = { .speed = I2C_SPEED_FAST, @@ -146,13 +146,17 @@ .sda_hold = 36, } }, + .i2c[5] = { + .speed = I2C_SPEED_FAST, + }, }"
device domain 0 on device pci 00.0 on end # Host Bridge device pci 02.0 on end # Integrated Graphics Device - device pci 04.0 off end # SA Thermal device - device pci 12.0 off end # Thermal Subsystem + device pci 04.0 off end # SA Thermal device + device pci 05.0 on end #IPU + device pci 12.0 off end # Thermal Subsystem device pci 12.5 off end # UFS SCS device pci 12.6 off end # GSPI #2 device pci 14.0 on @@ -285,8 +289,8 @@ device pci 16.4 off end # Management Engine Interface 3 device pci 16.5 off end # Management Engine Interface 4 device pci 17.0 off end # SATA - device pci 19.0 on end # I2C #4 - device pci 19.1 off end # I2C #5 + device pci 19.0 off end # I2C #4 Cam 0 + device pci 19.1 on end # I2C #5 Cam 1 and VCM device pci 19.2 on end # UART #2 device pci 1a.0 on end # eMMC device pci 1c.0 on end # PCI Express Port 1 @@ -308,7 +312,7 @@ device spi 0 on end end end # GSPI #1 - device pci 1f.0 on end # eSPI Interface + device pci 1f.0 on end # eSPI Interface device pci 1f.1 on end # P2SB device pci 1f.2 on end # Power Management Controller device pci 1f.3 on end # Intel HDA diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c index 5a127a6..d67f4a4 100644 --- a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/gpio.c @@ -64,13 +64,19 @@ /* I2C0_SCL */ PAD_CFG_NF(GPP_C17, UP_2K, DEEP, NF1),
+ /* CAM2_RST_N */ + PAD_CFG_GPO(GPP_C19, 0, PLTRST), + /* WIFI_RF_KILL_N */ PAD_CFG_GPO(GPP_D0, 1, PLTRST),
/* BT_RF_KILL_N */ PAD_CFG_GPO(GPP_D1, 1, PLTRST),
- /* LAN_RST_N */ + /* CAM2_PWREN */ + PAD_CFG_GPO(GPP_D4, 0, PLTRST), + + /*LAN_RST_N*/ PAD_CFG_GPO(GPP_D6, 1, PLTRST),
/* AVS_I2S_MCLK */ @@ -85,6 +91,15 @@ /* CNV_PA_BLANKING */ PAD_CFG_NF(GPP_D21, NONE, DEEP, NF1),
+ /* I2C5_SDA */ + PAD_CFG_NF(GPP_D22, NONE, PLTRST, NF1), + + /* I2C5_SCL */ + PAD_CFG_NF(GPP_D23, NONE, PLTRST, NF1), + + /* IMGCLKOUT_1 */ + PAD_CFG_NF(GPP_E2, NONE, PLTRST, NF1), + /* WWAN_FCP_OFF_N */ PAD_CFG_GPO(GPP_E3, 1, PLTRST),
diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl new file mode 100644 index 0000000..318b0de --- /dev/null +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/include/variant/acpi/camera.asl @@ -0,0 +1,3 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#include <baseboard/acpi/camera.asl>
9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39669 )
Change subject: mb/intel/jasperlake_rvp: Add world facing camera support ......................................................................
Patch Set 23:
Automatic boot test returned (PASS/FAIL/TOTAL): 4/0/4 Emulation targets: "QEMU x86 q35/ich9" using payload TianoCore : SUCCESS : https://lava.9esec.io/r/3958 "QEMU x86 q35/ich9" using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/3957 "QEMU x86 i440fx/piix4" using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/3956 "QEMU AArch64" using payload LinuxBoot_u-root_kexec : SUCCESS : https://lava.9esec.io/r/3955
Please note: This test is under development and might not be accurate at all!