Re: [PATCH] board: ls1046ardb: force PCI device enumeration

[Martin Schiller <ms@dev.tdt.de>]

On 2021-11-22 17:30, Camelia Alexandra Groza (OSS) wrote:
>> -----Original Message-----
>> From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Martin 
>> Schiller
>> Sent: Wednesday, November 17, 2021 13:59
>> To: u-boot@lists.denx.de
>> Cc: Martin Schiller <ms@dev.tdt.de>; Priyanka Jain 
>> <priyanka.jain@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@dev.tdt.de>
>> CC: Priyanka Jain <priyanka.jain@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?
> 

Hi

Yes, I can do that. But I can't test it due to lack of hardware.

> Acked-by: Camelia Groza <camelia.groza@nxp.com>