diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-04-05 13:02:45 +0200 |
|---|---|---|
| committer | Angel Pons <th3fanbus@gmail.com> | 2021-04-21 14:21:44 +0000 |
| commit | 0c0d49229db9ef68a5ea9267296e900ac7274da8 (patch) | |
| tree | 80982c6cb55bd90d9abb0fe82449c48d4d13854f /src/soc/intel | |
| parent | 73a22edcc8894c34df1234ae02d5318f18e3f7b8 (diff) | |
soc/intel: Replace open-coded buffer length calculation
Use `sizeof(value)` instead of manually calculating the buffer size.
Change-Id: Ibe49e40b1c4f2c0b661d94e59059a95bdb204197
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52107
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Diffstat (limited to 'src/soc/intel')
| -rw-r--r-- | src/soc/intel/alderlake/chip.c | 6 | ||||
| -rw-r--r-- | src/soc/intel/cannonlake/gpio_common.c | 6 | ||||
| -rw-r--r-- | src/soc/intel/elkhartlake/chip.c | 6 | ||||
| -rw-r--r-- | src/soc/intel/icelake/chip.c | 6 | ||||
| -rw-r--r-- | src/soc/intel/jasperlake/chip.c | 6 | ||||
| -rw-r--r-- | src/soc/intel/tigerlake/chip.c | 6 |
6 files changed, 12 insertions, 24 deletions
diff --git a/src/soc/intel/alderlake/chip.c b/src/soc/intel/alderlake/chip.c index dc54845600..f4d02412eb 100644 --- a/src/soc/intel/alderlake/chip.c +++ b/src/soc/intel/alderlake/chip.c @@ -116,11 +116,9 @@ static void soc_fill_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } diff --git a/src/soc/intel/cannonlake/gpio_common.c b/src/soc/intel/cannonlake/gpio_common.c index 2c2dcdbabd..4307260cd1 100644 --- a/src/soc/intel/cannonlake/gpio_common.c +++ b/src/soc/intel/cannonlake/gpio_common.c @@ -14,11 +14,9 @@ void soc_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } diff --git a/src/soc/intel/elkhartlake/chip.c b/src/soc/intel/elkhartlake/chip.c index 3917ea0af8..ad634819f2 100644 --- a/src/soc/intel/elkhartlake/chip.c +++ b/src/soc/intel/elkhartlake/chip.c @@ -105,11 +105,9 @@ static void soc_fill_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } diff --git a/src/soc/intel/icelake/chip.c b/src/soc/intel/icelake/chip.c index 32b1830096..05f42b4e7a 100644 --- a/src/soc/intel/icelake/chip.c +++ b/src/soc/intel/icelake/chip.c @@ -95,11 +95,9 @@ static void soc_fill_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } diff --git a/src/soc/intel/jasperlake/chip.c b/src/soc/intel/jasperlake/chip.c index ea29fd8b43..a8e3c88128 100644 --- a/src/soc/intel/jasperlake/chip.c +++ b/src/soc/intel/jasperlake/chip.c @@ -111,11 +111,9 @@ static void soc_fill_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } diff --git a/src/soc/intel/tigerlake/chip.c b/src/soc/intel/tigerlake/chip.c index 1affcce078..eefc408bc2 100644 --- a/src/soc/intel/tigerlake/chip.c +++ b/src/soc/intel/tigerlake/chip.c @@ -116,11 +116,9 @@ static void soc_fill_gpio_pm_configuration(void) const config_t *config = config_of_soc(); if (config->gpio_override_pm) - memcpy(value, config->gpio_pm, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memcpy(value, config->gpio_pm, sizeof(value)); else - memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(uint8_t) * - TOTAL_GPIO_COMM); + memset(value, MISCCFG_GPIO_PM_CONFIG_BITS, sizeof(value)); gpio_pm_configure(value, TOTAL_GPIO_COMM); } |