[PATCH] board: ls1046ardb: force PCI device enumeration

[Priyanka Jain (OSS)]

>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Camelia Alexandra
>Groza (OSS)
>Sent: Monday, November 22, 2021 10:01 PM
>To: Martin Schiller <ms at dev.tdt.de>; u-boot at lists.denx.de
>Cc: Priyanka Jain <priyanka.jain at nxp.com>
>Subject: RE: [PATCH] board: ls1046ardb: force PCI device enumeration
>
>> -----Original Message-----
>> From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Martin
>> Schiller
>> Sent: Wednesday, November 17, 2021 13:59
>> To: u-boot at lists.denx.de
>> Cc: Martin Schiller <ms at dev.tdt.de>; Priyanka Jain
>> <priyanka.jain at nxp.com>
>> Subject: [PATCH] board: ls1046ardb: force PCI device enumeration
>>
>> Commit 045ecf899252 ("configs: enable DM_ETH support for LS1046ARDB")
>> resulted in the PCI bus no longer being implicitly enumerated.
>>
>> However, this is necessary for the fdt pcie fixups to work.
>>
>> Therefore, similar to commit 8b6558bd4187 ("board: ls1088ardb:
>> transition to DM_ETH"), pci_init() is now called in the board_init()
>> routine when CONFIG_DM_ETH is active.
>>
>> Signed-off-by: Martin Schiller <ms at dev.tdt.de>
>> CC: Priyanka Jain <priyanka.jain at nxp.com>
>> ---
>>  board/freescale/ls1046ardb/ls1046ardb.c | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/board/freescale/ls1046ardb/ls1046ardb.c
>> b/board/freescale/ls1046ardb/ls1046ardb.c
>> index 93ef903f29..d0abfe8869 100644
>> --- a/board/freescale/ls1046ardb/ls1046ardb.c
>> +++ b/board/freescale/ls1046ardb/ls1046ardb.c
>> @@ -93,6 +93,10 @@ int board_init(void)
>>  	ppa_init();
>>  #endif
>>
>> +#if !defined(CONFIG_SYS_EARLY_PCI_INIT) && defined(CONFIG_DM_ETH)
>> +	pci_init();
>> +#endif
>> +
>>  	/* invert AQR105 IRQ pins polarity */
>>  	out_be32(&scfg->intpcr, AQR105_IRQ_MASK);
>>
>
>Hi
>
>Thank you for the patch. A very similar fix is needed for the ls1043ardb board.
>Are you interested to send a patch for that as well?
>
>Acked-by: Camelia Groza <camelia.groza at nxp.com>
Reviewed-by: Priyanka Jain <priyanka.jain at nxp.com>