diff options
| author | David Barbion <[email protected]> | 2025-09-10 23:18:55 +0200 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-11-02 12:15:23 -0600 |
| commit | 60d8819d2f8eeecd05b5b9576ac0f7d45490d908 (patch) | |
| tree | fd8833be0b2b3cab020d00c3f3ed95051a475e5c /board/anbernic | |
| parent | eb92fd0831e073a401954ada3efe59a81137e6d9 (diff) | |
board: rockchip: Fix RG353M model renaming
Anbernic RG353M is hardware compatible with RG353P. Only the form-factor
differs. So only one DTS is created for both machines with
"Anbernic RG353P" as default model. If a RG353M is detected, the model
should be overwritten with the correct name.
Actually, it's overwritten with "Anbernic" only making the process of
machine detection a little harder.
However, to determine the size of the string "Anbernic RG353M", it is
sizeof() which is used resulting in obtaining the size of the pointer
(which is 8 bytes on ARM64) not the size of the pointed string.
strlen() should be used instead.
Signed-off-by: David Barbion <[email protected]>
Reviewed-by: Kever Yang <[email protected]>
Diffstat (limited to 'board/anbernic')
| -rw-r--r-- | board/anbernic/rgxx3_rk3566/rgxx3-rk3566.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/anbernic/rgxx3_rk3566/rgxx3-rk3566.c b/board/anbernic/rgxx3_rk3566/rgxx3-rk3566.c index c1d1826fd14..d402374e90e 100644 --- a/board/anbernic/rgxx3_rk3566/rgxx3-rk3566.c +++ b/board/anbernic/rgxx3_rk3566/rgxx3-rk3566.c @@ -695,7 +695,7 @@ int ft_board_setup(void *blob, struct bd_info *bd) if (gd->board_type == RG353M) fdt_setprop(blob, 0, "model", rg3xx_model_details[RG353M].board_name, - sizeof(rg3xx_model_details[RG353M].board_name)); + strlen(rg3xx_model_details[RG353M].board_name)); if (rg3xx_model_details[gd->board_type].detect_panel) { ret = rgxx3_panel_fixup(blob); |