diff options
| author | Jan Samek <jan.samek@siemens.com> | 2023-01-27 11:02:20 +0100 |
|---|---|---|
| committer | Paul Fagerburg <pfagerburg@chromium.org> | 2023-02-02 15:53:37 +0000 |
| commit | 671cd1d16bcb83069cac810ca2df4f2b4d056491 (patch) | |
| tree | 42830e129bba2d3dd0ca12247cf4dd4ac769201b /src/drivers/i2c | |
| parent | f564c8b34abbc7010871cfa426af7ee7261c3fde (diff) | |
| download | coreboot-671cd1d16bcb83069cac810ca2df4f2b4d056491.tar.gz coreboot-671cd1d16bcb83069cac810ca2df4f2b4d056491.tar.bz2 coreboot-671cd1d16bcb83069cac810ca2df4f2b4d056491.zip | |
drv/i2c/ptn3460: Use PTN_EDID_LEN instead of constant
Contents of the EDID are passed by a reference to an array
of length 0x80, for which the macro 'PTN_EDID_LEN' has already
been around.
This patch makes use of this macro within the driver and mainboard
implementation utilizing it.
BUG=none
TEST=A successful build of mc_apl{1,4,5,7} and mc_ehl3 mainboards.
Change-Id: If7d254aaf45d717133bb426bd08f8f9fe5c05962
Signed-off-by: Jan Samek <jan.samek@siemens.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72653
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Mario Scheithauer <mario.scheithauer@siemens.com>
Diffstat (limited to 'src/drivers/i2c')
| -rw-r--r-- | src/drivers/i2c/ptn3460/ptn3460.c | 2 | ||||
| -rw-r--r-- | src/drivers/i2c/ptn3460/ptn3460.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/drivers/i2c/ptn3460/ptn3460.c b/src/drivers/i2c/ptn3460/ptn3460.c index 31db8ad7fe64..d2df04d21623 100644 --- a/src/drivers/i2c/ptn3460/ptn3460.c +++ b/src/drivers/i2c/ptn3460/ptn3460.c @@ -123,7 +123,7 @@ static void ptn3460_init(struct device *dev) init_done = true; } -__weak enum cb_err mb_get_edid(uint8_t edid_data[0x80]) +__weak enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN]) { return CB_ERR; } diff --git a/src/drivers/i2c/ptn3460/ptn3460.h b/src/drivers/i2c/ptn3460/ptn3460.h index 5d867cdd46bb..ecd2191f3922 100644 --- a/src/drivers/i2c/ptn3460/ptn3460.h +++ b/src/drivers/i2c/ptn3460/ptn3460.h @@ -52,7 +52,7 @@ struct ptn_3460_flash { /* We need functions which we can call to get mainboard specific data */ /* These functions can be implemented somewhere else but must exist. */ -extern enum cb_err mb_get_edid(uint8_t edid_data[0x80]); +extern enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN]); extern uint8_t mb_select_edid_table(void); extern enum cb_err mb_adjust_cfg(struct ptn_3460_config *cfg_ptr); |