diff mbox series

[2/2] board: dh_stm32mp1: Only print board code with CONFIG_SPL_DISPLAY_PRINT

Message ID 20230927124625.247858-1-hws@denx.de
State Accepted
Commit 07cdd22c38608e2f57d4612f20c8990ecf6c889b
Delegated to: Patrice Chotard
Headers show
Series [1/2] ram: stm32mp1: Only print RAM config with CONFIG_SPL_DISPLAY_PRINT | expand

Commit Message

Harald Seiler Sept. 27, 2023, 12:46 p.m. UTC
Ensure that the SoM and board code information is only printed when
CONFIG_SPL_DISPLAY_PRINT is set.

Signed-off-by: Harald Seiler <hws@denx.de>
---
 board/dhelectronics/dh_stm32mp1/board.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Marek Vasut Sept. 30, 2023, 9:02 p.m. UTC | #1
On 9/27/23 14:46, Harald Seiler wrote:
> Ensure that the SoM and board code information is only printed when
> CONFIG_SPL_DISPLAY_PRINT is set.
> 
> Signed-off-by: Harald Seiler <hws@denx.de>

Reviewed-by: Marek Vasut <marex@denx.de>
Patrice CHOTARD Oct. 4, 2023, 9:19 a.m. UTC | #2
On 9/30/23 23:02, Marek Vasut wrote:
> On 9/27/23 14:46, Harald Seiler wrote:
>> Ensure that the SoM and board code information is only printed when
>> CONFIG_SPL_DISPLAY_PRINT is set.
>>
>> Signed-off-by: Harald Seiler <hws@denx.de>
> 
> Reviewed-by: Marek Vasut <marex@denx.de>


Apply on stm32/next

Thanks
Marek Vasut Oct. 4, 2023, 11:41 a.m. UTC | #3
On 10/4/23 11:19, Patrice CHOTARD wrote:
> 
> 
> On 9/30/23 23:02, Marek Vasut wrote:
>> On 9/27/23 14:46, Harald Seiler wrote:
>>> Ensure that the SoM and board code information is only printed when
>>> CONFIG_SPL_DISPLAY_PRINT is set.
>>>
>>> Signed-off-by: Harald Seiler <hws@denx.de>
>>
>> Reviewed-by: Marek Vasut <marex@denx.de>
> 
> 
> Apply on stm32/next

This all should be on master, since we're way before 2024.01-rc1 now , 
2023.10 was just released on Monday.
Patrice CHOTARD Oct. 4, 2023, 1:39 p.m. UTC | #4
On 10/4/23 13:41, Marek Vasut wrote:
> On 10/4/23 11:19, Patrice CHOTARD wrote:
>>
>>
>> On 9/30/23 23:02, Marek Vasut wrote:
>>> On 9/27/23 14:46, Harald Seiler wrote:
>>>> Ensure that the SoM and board code information is only printed when
>>>> CONFIG_SPL_DISPLAY_PRINT is set.
>>>>
>>>> Signed-off-by: Harald Seiler <hws@denx.de>
>>>
>>> Reviewed-by: Marek Vasut <marex@denx.de>
>>
>>
>> Apply on stm32/next
> 
> This all should be on master, since we're way before 2024.01-rc1 now , 2023.10 was just released on Monday.

Hi Marek

Yes it was my intention, thanks for checking ;-)

Patrice
Marek Vasut Oct. 4, 2023, 1:40 p.m. UTC | #5
On 10/4/23 15:39, Patrice CHOTARD wrote:
> 
> 
> On 10/4/23 13:41, Marek Vasut wrote:
>> On 10/4/23 11:19, Patrice CHOTARD wrote:
>>>
>>>
>>> On 9/30/23 23:02, Marek Vasut wrote:
>>>> On 9/27/23 14:46, Harald Seiler wrote:
>>>>> Ensure that the SoM and board code information is only printed when
>>>>> CONFIG_SPL_DISPLAY_PRINT is set.
>>>>>
>>>>> Signed-off-by: Harald Seiler <hws@denx.de>
>>>>
>>>> Reviewed-by: Marek Vasut <marex@denx.de>
>>>
>>>
>>> Apply on stm32/next
>>
>> This all should be on master, since we're way before 2024.01-rc1 now , 2023.10 was just released on Monday.
> 
> Hi Marek
> 
> Yes it was my intention, thanks for checking ;-)

You're welcome.
diff mbox series

Patch

diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board/dhelectronics/dh_stm32mp1/board.c
index f9cfabe2420..b933761d0de 100644
--- a/board/dhelectronics/dh_stm32mp1/board.c
+++ b/board/dhelectronics/dh_stm32mp1/board.c
@@ -229,8 +229,9 @@  static void board_get_coding_straps(void)
 
 	gpio_free_list_nodev(gpio, ret);
 
-	printf("Code:  SoM:rev=%d,ddr3=%d Board:rev=%d\n",
-		somcode, ddr3code, brdcode);
+	if (CONFIG_IS_ENABLED(DISPLAY_PRINT))
+		printf("Code:  SoM:rev=%d,ddr3=%d Board:rev=%d\n",
+		       somcode, ddr3code, brdcode);
 }
 
 int board_stm32mp1_ddr_config_name_match(struct udevice *dev,