[PATCH] ARM: dts: stm32: fix display pinmux for stm32f746-disco

Patrice CHOTARD patrice.chotard at foss.st.com
Mon Jun 5 14:20:09 CEST 2023


Hi Tom

On 6/1/23 19:11, Tom Rini wrote:
> On Thu, Jun 01, 2023 at 07:06:02PM +0200, Dario Binacchi wrote:
>> As reported by the datasheet (DocID027590 Rev 4) for PG12:
>> - AF9  -> LCD_B4
>> - AF14 -> LCD_B1
>>
>> So replace AF14 with AF9 for PG12 in the dts.
>>
>> Fixes: fe63d3cfb77ef ("ARM: dts: stm32: Sync DT with v4.20 kernel for stm32f7")
>> Signed-off-by: Dario Binacchi <dario.binacchi at amarulasolutions.com>
>>
>> ---
>>
>>  arch/arm/dts/stm32f746-disco-u-boot.dtsi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32f746-disco-u-boot.dtsi b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
>> index 19b5451db441..522cffb1ac9f 100644
>> --- a/arch/arm/dts/stm32f746-disco-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
>> @@ -169,7 +169,7 @@
>>  	ltdc_pins: ltdc at 0 {
>>  		pins {
>>  			pinmux = <STM32_PINMUX('E', 4, AF14)>, /* B0 */
>> -				 <STM32_PINMUX('G',12, AF14)>, /* B4 */
>> +				 <STM32_PINMUX('G',12, AF9)>,  /* B4 */
>>  				 <STM32_PINMUX('I', 9, AF14)>, /* VSYNC */
>>  				 <STM32_PINMUX('I',10, AF14)>, /* HSYNC */
>>  				 <STM32_PINMUX('I',14, AF14)>, /* CLK */
> 
> Why is this whole stanza in a -u-boot.dtsi file and not an upstream
> file?
> 

LTDC entry has been put in stm32f746-disco-u-boot.dtsi by anticipation as described
in commit 01aabf97d1f0 ("ARM: dts: stm32: Migrate U-boot nodes to U-boot DT files for stm32f7")

So U-Boot DT will be synchronized with kernel one when Dario series will be available in kernel release.
The LTDC entry located in stm32f746-disco-u-boot.dtsi will be then removed.

Thanks
Patrice




More information about the U-Boot mailing list