From c3a83bf51426ceb3176183479c9edab212d82515 Mon Sep 17 00:00:00 2001
From: Matt Delco <delco@chromium.org>
Date: Tue, 16 Jun 2020 12:02:34 +0530
Subject: [PATCH] drivers/intel/mipi_camera: Handle NVM and VCM device type

This change adds support in mipi_camera driver to handle NVM and VCM
device types.

Change-Id: I24cb7f010d89bc8d14e0b4c8fe693ba6e9c68941
Signed-off-by: Sugnan Prabhu S <sugnan.prabhu.s@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42466
Reviewed-by: Rizwan Qureshi <rizwan.qureshi@intel.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/drivers/intel/mipi_camera/camera.c | 51 ++++++++++++++++++++++++++
 src/drivers/intel/mipi_camera/chip.h   | 13 +++++++
 2 files changed, 64 insertions(+)

diff --git a/src/drivers/intel/mipi_camera/camera.c b/src/drivers/intel/mipi_camera/camera.c
index 4356b0a602..b1075b85ed 100644
--- a/src/drivers/intel/mipi_camera/camera.c
+++ b/src/drivers/intel/mipi_camera/camera.c
@@ -10,6 +10,46 @@
 #include <device/pci_def.h>
 #include "chip.h"
 
+static void camera_fill_nvm(const struct device *dev)
+{
+	struct drivers_intel_mipi_camera_config *config = dev->chip_info;
+	struct acpi_dp *dsd = acpi_dp_new_table("_DSD");
+
+	/* It might be possible to default size or width based on type. */
+	if (!config->disable_nvm_defaults && !config->nvm_pagesize)
+		config->nvm_pagesize = 1;
+
+	if (!config->disable_nvm_defaults && !config->nvm_readonly)
+		config->nvm_readonly = 1;
+
+	if (config->nvm_size)
+		acpi_dp_add_integer(dsd, "size", config->nvm_size);
+
+	if (config->nvm_pagesize)
+		acpi_dp_add_integer(dsd, "pagesize", config->nvm_pagesize);
+
+	if (config->nvm_readonly)
+		acpi_dp_add_integer(dsd, "read-only", config->nvm_readonly);
+
+	if (config->nvm_width)
+		acpi_dp_add_integer(dsd, "address-width", config->nvm_width);
+
+	acpi_dp_write(dsd);
+}
+
+static void camera_fill_vcm(const struct device *dev)
+{
+	struct drivers_intel_mipi_camera_config *config = dev->chip_info;
+	struct acpi_dp *dsd;
+
+	if (!config->vcm_compat)
+		return;
+
+	dsd = acpi_dp_new_table("_DSD");
+	acpi_dp_add_string(dsd, "compatible", config->vcm_compat);
+	acpi_dp_write(dsd);
+}
+
 static void write_pci_camera_device(const struct device *dev)
 {
 	struct drivers_intel_mipi_camera_config *config = dev->chip_info;
@@ -93,6 +133,17 @@ static void write_camera_device_common(const struct device *dev)
 	    config->device_type == INTEL_ACPI_CAMERA_VCM) {
 		acpigen_write_name_integer("CAMD", config->device_type);
 	}
+
+	switch (config->device_type) {
+	case INTEL_ACPI_CAMERA_VCM:
+		camera_fill_vcm(dev);
+		break;
+	case INTEL_ACPI_CAMERA_NVM:
+		camera_fill_nvm(dev);
+		break;
+	default:
+		break;
+	}
 }
 
 static void camera_fill_ssdt(const struct device *dev)
diff --git a/src/drivers/intel/mipi_camera/chip.h b/src/drivers/intel/mipi_camera/chip.h
index c029570064..b04a10013e 100644
--- a/src/drivers/intel/mipi_camera/chip.h
+++ b/src/drivers/intel/mipi_camera/chip.h
@@ -95,6 +95,19 @@ struct drivers_intel_mipi_camera_config {
 	const char *vcm_name;		/* defaults to |vcm_address| device */
 	uint16_t rom_address;		/* I2C to use if ssdb.rom_type != 0 */
 	uint16_t vcm_address;		/* I2C to use if ssdb.vcm_type != 0 */
+	/*
+	 * Settings specific to nvram. Many values, if left as zero, will be assigned a default.
+	 * Set disable_nvm_defaults to non-zero if you want to disable the defaulting behavior
+	 * so you can use zero for a value.
+	 */
+	bool disable_nvm_defaults;
+	uint32_t nvm_size;
+	uint32_t nvm_pagesize;
+	uint32_t nvm_readonly;
+	uint32_t nvm_width;
+
+	/* Settings specific to vcm */
+	const char *vcm_compat;
 };
 
 #endif