[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus: Add variant for BIP

Naveen Manohar (Code Review) gerrit at coreboot.org
Thu May 10 05:57:38 CEST 2018


Naveen Manohar has uploaded this change for review. ( https://review.coreboot.org/26211


Change subject: mb/google/octopus: Add variant for BIP
......................................................................

mb/google/octopus: Add variant for BIP

Add a new variant of Octopus for the BIP board.

BUG=b:77892150
TEST=Yet to be verified on a PO BIP board.
Only Audio entries are added, other peripherals are
yet to be configured.

Change-Id: Iee91518c03a0e9e6ed52bc54a60fc607730a0b7d
Signed-off-by: Naveen Manohar <naveen.m at intel.com>
---
M src/mainboard/google/octopus/Kconfig
A src/mainboard/google/octopus/variants/bip/devicetree.cb
A src/mainboard/google/octopus/variants/bip/nhlt.c
3 files changed, 262 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/11/26211/1

diff --git a/src/mainboard/google/octopus/Kconfig b/src/mainboard/google/octopus/Kconfig
index 3254ec6..3679a5c 100644
--- a/src/mainboard/google/octopus/Kconfig
+++ b/src/mainboard/google/octopus/Kconfig
@@ -35,6 +35,7 @@
 
 config MAINBOARD_DIR
 	string
+	default "bip" if BOARD_GOOGLE_BIP
 	default google/octopus
 
 config VARIANT_DIR
@@ -47,6 +48,7 @@
 config DEVICETREE
 	string
 	default "variants/baseboard/devicetree.cb"
+	default "variants/bip/devicetree.cb" if BOARD_GOOGLE_BIP
 
 config MAINBOARD_PART_NUMBER
 	string
@@ -58,6 +60,7 @@
 config MAINBOARD_FAMILY
 	string
 	default "Google_Octopus"
+	default "Google_Bip" if BOARD_GOOGLE_BIP
 
 config GBB_HWID
 	string
@@ -89,6 +92,12 @@
 	select NHLT_DA7219
 	select NHLT_MAX98357
 
+config INCLUDE_NHLT_BLOBS_BIP
+	bool "Include blobs for bip audio."
+	select NHLT_DMIC_4CH_16B
+	select NHLT_MAX98357
+	select NHLT_RT5682
+
 config DRIVER_TPM_SPI_BUS
 	default 0x1
 
diff --git a/src/mainboard/google/octopus/variants/bip/devicetree.cb b/src/mainboard/google/octopus/variants/bip/devicetree.cb
new file mode 100644
index 0000000..815594c
--- /dev/null
+++ b/src/mainboard/google/octopus/variants/bip/devicetree.cb
@@ -0,0 +1,203 @@
+chip soc/intel/apollolake
+	device cpu_cluster 0 on
+		device lapic 0 on end
+	end
+
+	register "pcie_rp_clkreq_pin[2]" = "3"    # wifi/bt
+	# Disable unused clkreq of PCIe root ports
+	register "pcie_rp_clkreq_pin[0]" = "CLKREQ_DISABLED"
+	register "pcie_rp_clkreq_pin[1]" = "CLKREQ_DISABLED"
+	register "pcie_rp_clkreq_pin[3]" = "CLKREQ_DISABLED"
+	register "pcie_rp_clkreq_pin[4]" = "CLKREQ_DISABLED"
+	register "pcie_rp_clkreq_pin[5]" = "CLKREQ_DISABLED"
+
+	# Set de-emphasis to disabled for PCIE WiFI (Thunderpeak)
+	# as it is required for detection
+	register "pcie_rp_deemphasis_enable[2]" = "0"
+	# Set de-emphasis to default (enabled) for remaining ports
+	register "pcie_rp_deemphasis_enable[0]" = "1"
+	register "pcie_rp_deemphasis_enable[1]" = "1"
+	register "pcie_rp_deemphasis_enable[3]" = "1"
+	register "pcie_rp_deemphasis_enable[4]" = "1"
+	register "pcie_rp_deemphasis_enable[5]" = "1"
+
+	# GPIO for PERST_0 (WLAN_PE_RST)
+	register "prt0_gpio" = "GPIO_164"
+
+	# GPE configuration
+	# Note that GPE events called out in ASL code rely on this
+	# route, i.e., if this route changes then the affected GPE
+	# offset bits also need to be changed. This sets the PMC register
+	# GPE_CFG fields.
+	register "gpe0_dw1" = "PMC_GPE_NW_63_32"
+	register "gpe0_dw2" = "PMC_GPE_N_95_64"
+	register "gpe0_dw3" = "PMC_GPE_NW_31_0"
+
+	# PL1 override 8000 mW: Due to error in the energy calculation for
+	# current VR solution. Experiments show that SoC TDP max (6W) can
+	# be reached when RAPL PL1 is set to 8W.
+	register "tdp_pl1_override_mw" = "8000"
+	# Set RAPL PL2 to 15W.
+	register "tdp_pl2_override_mw" = "15000"
+
+	# Minimum SLP S3 assertion width 28ms.
+	register "slp_s3_assertion_width_usecs" = "28000"
+
+	# Enable lpss s0ix
+	register "lpss_s0ix_enable" = "1"
+
+	# Enable DPTF
+	register "dptf_enable" = "1"
+
+	# Enable Audio Clock and Power gating
+	register "hdaudio_clk_gate_enable" = "1"
+	register "hdaudio_pwr_gate_enable" = "1"
+	register "hdaudio_bios_config_lockdown" = "1"
+
+	# digitizer at 400kHz
+	register "i2c[0]" = "{
+		.speed = I2C_SPEED_FAST,
+		.rise_time_ns = 152,
+		.fall_time_ns = 30,
+	}"
+
+	# Enable I2C5 for audio codec at 400kHz
+	register "i2c[5]" = "{
+		.speed = I2C_SPEED_FAST,
+		.rise_time_ns = 104,
+		.fall_time_ns = 52,
+	}"
+
+	# trackpad at 400kHz
+	register "i2c[6]" = "{
+		.speed = I2C_SPEED_FAST,
+		.rise_time_ns = 114,
+		.fall_time_ns = 164,
+		.data_hold_time_ns = 350,
+	}"
+
+	# touchscreen at 400kHz
+	register "i2c[7]" = "{
+		.speed = I2C_SPEED_FAST,
+		.rise_time_ns = 76,
+		.fall_time_ns = 164,
+	}"
+
+	# Use GSPI0 for cr50 TPM. Early init is required to set up a BAR for TPM
+	# communication before memory is up.
+	register "gspi[0]" = "{
+		.speed_mhz = 1,
+		.early_init = 1,
+	}"
+
+	register "pnp_settings" = "PNP_PERF_POWER"
+
+	device domain 0 on
+		device pci 00.0 on  end	# - Host Bridge
+		device pci 00.1 on  end	# - DPTF
+		device pci 00.2 off end	# - NPK
+		device pci 02.0 on  end	# - Gen
+		device pci 03.0 on  end	# - Iunit
+		device pci 0c.0 on  end	# - CNVi
+		device pci 0d.0 on  end	# - P2SB
+		device pci 0d.1 on  end	# - PMC
+		device pci 0d.2 on  end	# - Fast SPI
+		device pci 0d.3 on  end	# - Shared SRAM
+		device pci 0e.0 on
+			chip drivers/generic/max98357a
+				register "sdmode_gpio" =  "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_91)"
+				register "sdmode_delay" = "5"
+				device generic 0 on end
+			end
+		end	# - Audio
+		device pci 0f.0 on  end	# - Heci1
+		device pci 0f.1 on  end	# - Heci2
+		device pci 0f.2 on  end	# - Heci3
+		device pci 11.0 off end	# - ISH
+		device pci 12.0 off end	# - SATA
+		device pci 13.0 on
+			chip drivers/intel/wifi
+				device pci 00.0 on end
+			end
+		end	# - PCIe-A 0 Onboard M2 Slot(Wifi)
+		device pci 13.1 off end	# - PCIe-A 1
+		device pci 13.2 off end	# - PCIe-A 2
+		device pci 13.3 off end	# - PCIe-A 3
+		device pci 14.0 off end	# - PCIe-B 0
+		device pci 14.1 off end	# - PCIe-B 1
+		device pci 15.0 on  end	# - XHCI
+		device pci 15.1 off end	# - XDCI
+		device pci 16.0 on
+			chip drivers/i2c/hid
+				register "generic.hid" = ""WCOM50C1""
+				register "generic.desc" = ""WCOM Digitizer""
+				register "generic.irq" = "ACPI_IRQ_LEVEL_LOW(GPIO_139_IRQ)"
+				register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_140)"
+				register "generic.has_power_resource" = "1"
+				register "hid_desc_reg_offset" = "0x1"
+				device i2c 0x9 on end
+			end
+		end	# - I2C 0
+		device pci 16.1 on end	# - I2C 1
+		device pci 16.2 on end	# - I2C 2
+		device pci 16.3 on end	# - I2C 3
+		device pci 17.0 on end	# - I2C 4
+		device pci 17.1 on
+			chip drivers/i2c/generic
+				register "hid" = ""10EC5682""
+				register "name" = ""RT58""
+				register "desc" = ""Realtek RT5682""
+				register "irq" = "ACPI_IRQ_LEVEL_LOW(GPIO_20_IRQ)"
+				register "probed" = "1"
+				device i2c 1a on end
+			end
+		end	# - I2C 5
+		device pci 17.2 on
+			chip drivers/i2c/generic
+				register "hid" = ""ELAN0000""
+				register "desc" = ""ELAN Touchpad""
+				register "irq" = "ACPI_IRQ_EDGE_LOW(GPIO_135_IRQ)"
+				register "wake" = "GPE0_DW2_02"
+				register "probed" = "1"
+				device i2c 15 on end
+			end
+		end	# - I2C 6
+		device pci 17.3 on
+			chip drivers/i2c/generic
+				register "hid" = ""ELAN0001""
+				register "desc" = ""ELAN Touchscreen""
+				register "irq" = "ACPI_IRQ_EDGE_LOW(GPIO_212_IRQ)"
+				register "probed" = "1"
+				register "reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_105)"
+				register "reset_delay_ms" = "20"
+				register "enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPIO_213)"
+				register "enable_delay_ms" = "1"
+				register "has_power_resource" = "1"
+				device i2c 10 on end
+			end
+		end 	# - I2C 7
+		device pci 18.0 on  end	# - UART 0
+		device pci 18.1 off end	# - UART 1
+		device pci 18.2 on  end	# - UART 2
+		device pci 18.3 off end	# - UART 3
+		device pci 19.0 on
+			chip drivers/spi/acpi
+				register "hid" = "ACPI_DT_NAMESPACE_HID"
+				register "compat_string" = ""google,cr50""
+				register "irq" = "ACPI_IRQ_EDGE_LOW(GPIO_63_IRQ)"
+				device spi 0 on end
+			end
+		end # - GSPI 0
+		device pci 19.1 off end	# - SPI 1
+		device pci 19.2 on  end	# - SPI 2
+		device pci 1a.0 on  end	# - PWM
+		device pci 1c.0 on  end	# - eMMC
+		device pci 1e.0 off end	# - SDIO
+		device pci 1f.0 on
+			chip ec/google/chromeec
+				device pnp 0c09.0 on end
+			end
+		end	# - ESPI
+		device pci 1f.1 on  end	# - SMBUS
+	end
+end
diff --git a/src/mainboard/google/octopus/variants/bip/nhlt.c b/src/mainboard/google/octopus/variants/bip/nhlt.c
new file mode 100644
index 0000000..b5e2127
--- /dev/null
+++ b/src/mainboard/google/octopus/variants/bip/nhlt.c
@@ -0,0 +1,50 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2018 Intel Corp.
+ *
+ * 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/variants.h>
+#include <compiler.h>
+#include <console/console.h>
+#include <nhlt.h>
+#include <soc/nhlt.h>
+
+void __weak variant_nhlt_init(struct nhlt *nhlt)
+{
+	/* 2 Channel DMIC array. */
+	if (!nhlt_soc_add_dmic_array(nhlt, 2))
+		printk(BIOS_ERR, "Added 2CH DMIC array.\n");
+
+	/* 4 Channel DMIC array. */
+	if (!nhlt_soc_add_dmic_array(nhlt, 4))
+		printk(BIOS_ERR, "Added 4CH DMIC arrays.\n");
+
+	/* Realtek for Headset codec.
+	 * Headset codec is bi-directional but uses the same configuration
+	 * settings for render and capture endpoints.
+	 */
+	if (!nhlt_soc_add_rt5682(nhlt, AUDIO_LINK_SSP2))
+		printk(BIOS_ERR, "Added Realtek_5682 codec.\n");
+
+	/* MAXIM Smart Amps for left and right speakers. */
+	if (!nhlt_soc_add_max98357(nhlt, AUDIO_LINK_SSP1))
+		printk(BIOS_ERR, "Added Maxim_98357 codec.\n");
+}
+
+void variant_nhlt_oem_overrides(const char **oem_id, const char **oem_table_id,
+				uint32_t *oem_revision)
+{
+	*oem_id = "GOOGLE";
+	*oem_table_id = "BIPMAX";
+	*oem_revision = 0;
+}

-- 
To view, visit https://review.coreboot.org/26211
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee91518c03a0e9e6ed52bc54a60fc607730a0b7d
Gerrit-Change-Number: 26211
Gerrit-PatchSet: 1
Gerrit-Owner: Naveen Manohar <naveen.m at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180510/b55dab9b/attachment-0001.html>


More information about the coreboot-gerrit mailing list