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>
This commit is contained in:
Jan Samek 2023-01-27 11:02:20 +01:00 committed by Paul Fagerburg
parent f564c8b34a
commit 671cd1d16b
7 changed files with 7 additions and 7 deletions

View File

@ -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;
}

View File

@ -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);

View File

@ -10,7 +10,7 @@
* @param edid_data pointer to EDID data in driver
* @return CB_SUCCESS on successful EDID data retrieval, CB_ERR otherwise
*/
enum cb_err mb_get_edid(uint8_t edid_data[0x80])
enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN])
{
const char *hwi_block = "hwinfo.hex";

View File

@ -10,7 +10,7 @@
* @param edid_data pointer to EDID data in driver
* @return CB_SUCCESS on successful EDID data retrieval, CB_ERR otherwise
*/
enum cb_err mb_get_edid(uint8_t edid_data[0x80])
enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN])
{
const char *hwi_block = "hwinfo.hex";

View File

@ -33,7 +33,7 @@ static void igd_disable(void)
* @param edid_data pointer to EDID data in driver
* @return CB_SUCCESS on successful EDID data retrieval, CB_ERR otherwise
*/
enum cb_err mb_get_edid(uint8_t edid_data[0x80])
enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN])
{
const char *hwi_block = "hwinfo.hex";

View File

@ -10,7 +10,7 @@
* @param edid_data pointer to EDID data in driver
* @return CB_SUCCESS on successful EDID data retrieval, CB_ERR otherwise
*/
enum cb_err mb_get_edid(uint8_t edid_data[0x80])
enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN])
{
const char *hwi_block = "hwinfo.hex";

View File

@ -10,7 +10,7 @@
* @param edid_data pointer to EDID data in driver
* @return CB_SUCCESS on successful EDID data retrieval, CB_ERR otherwise
*/
enum cb_err mb_get_edid(uint8_t edid_data[0x80])
enum cb_err mb_get_edid(uint8_t edid_data[PTN_EDID_LEN])
{
const char *hwi_block = "hwinfo.hex";