erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

mb/google/geralt: Adjust VSP/VSN voltage for CSOT_PNA957QT1_1 panel

Browse source 
The system default VSP/VSN voltage for Ciri is +-5.7V, which causes
the Gray screen pattern to display abnormally with screen corruption.

According to CSOT panel vendor's requirement, VSP/VSN for the
CSOT_PNA957QT1_1 panel needs to be adjusted to +-6V. So modify the
relevant register values accordingly.

BUG=b:399728328
TEST=Boot to firmware screen and kernel, measure the voltage.
BRANCH=geralt

Change-Id: I1b69303317f5ef47818f4a6a0c851bf650285e51
Signed-off-by: Yang Wu <wuyang5@huaqin.corp-partner.google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/86640
Reviewed-by: Yu-Ping Wu <yupingso@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Yidi Lin <yidilin@google.com>
This commit is contained in:
Yang Wu 2025-02-26 21:04:51 +08:00 • committed by Yidi Lin
commit 5c4f49708c
1 changed files with 18 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

24
src/mainboard/google/geralt/panel_ciri.c
View file

@ -7,12 +7,14 @@
#include "panel.h"
#define PMIC_TPS65132_I2C I2C3
#define VOLTAGE_5_7V 0x11
#define VOLTAGE_6V 0x14
static void power_on_mipi_himax_83102j(void)
static void power_on_mipi_himax_83102j(uint8_t voltage_reg_val)
{
const struct tps65132s_reg_setting reg_settings[] = {
{ PMIC_TPS65132_VPOS, 0x11, 0x1f },
{ PMIC_TPS65132_VNEG, 0x11, 0x1f },
{ PMIC_TPS65132_VPOS, voltage_reg_val, 0x1f },
{ PMIC_TPS65132_VNEG, voltage_reg_val, 0x1f },
{ PMIC_TPS65132_DLYX, 0x55, 0xff },
{ PMIC_TPS65132_ASSDD, 0x5b, 0xff },
};
@ -27,10 +29,20 @@ static void power_on_mipi_himax_83102j(void)
power_on_mipi_panel(&cfg);
}
static void power_on_mipi_himax_83102j_5_7v(void)
{
power_on_mipi_himax_83102j(VOLTAGE_5_7V);
}
static void power_on_mipi_himax_83102j_6v(void)
{
power_on_mipi_himax_83102j(VOLTAGE_6V);
}
static struct panel_description ciri_panels[] = {
[17] = {
.name = "BOE_NV110WUM_L60",
.power_on = power_on_mipi_himax_83102j,
.power_on = power_on_mipi_himax_83102j_5_7v,
.configure_backlight = configure_mipi_pwm_backlight,
.orientation = LB_FB_ORIENTATION_LEFT_UP,
.disp_path = DISP_PATH_MIPI,
@ -39,7 +51,7 @@ static struct panel_description ciri_panels[] = {
[34] = {
.name = "IVO_T109NW41",
.power_on = power_on_mipi_himax_83102j,
.power_on = power_on_mipi_himax_83102j_5_7v,
.configure_backlight = configure_mipi_pwm_backlight,
.orientation = LB_FB_ORIENTATION_LEFT_UP,
.disp_path = DISP_PATH_MIPI,
@ -48,7 +60,7 @@ static struct panel_description ciri_panels[] = {
[51] = {
.name = "CSOT_PNA957QT1_1",
.power_on = power_on_mipi_himax_83102j,
.power_on = power_on_mipi_himax_83102j_6v,
.configure_backlight = configure_mipi_pwm_backlight,
.orientation = LB_FB_ORIENTATION_LEFT_UP,
.disp_path = DISP_PATH_MIPI,