Re: [PATCH 2/2] imx8mm_evk: Add Serial Download Protocol support

Peng Fan
2022-09-21T05:26:00Z

On 9/20/2022 7:13 PM, Fabio Estevam wrote:
> Hi Peng,
>
> On Tue, Sep 20, 2022 at 3:56 AM Peng Fan <peng.fan@oss.nxp.com> wrote:
>>
>> Hi Fabio,
>>
>> Just have a question, the tcpci driver is not supported, so how
>> do you manage to make SDP work?
>
> I understand that the tcpci driver is required when a USB role switch is needed.
>
> I am only using the usbotg1 as a peripheral.

ok, got it.

>
> Here are the logs:
>
> On the host PC:
>
> $ sudo uuu -brun spl flash.bin
>
> On the console:
>
> U-Boot SPL 2022.10-rc5-00003-g9e02955991-dirty (Sep 19 2022 - 21:49:01 -0300)
> No pmic
> SEC0: RNG instantiated
> Normal Boot
> WDT: Started watchdog@30280000 with servicing (60s timeout)
> Trying to boot from USB SDP
> SDP: initialize...
> SDP: handle requests...
> Downloading file of size 885712 to 0x40400000... done
> Jumping to header at 0x40400000
> Header Tag is not an IMX image
> Found header at 0x40420400
> NOTICE: BL31: v2.2(release):rel_imx_5.4.47_2.2.0-0-gc949a888e
> NOTICE: BL31: Built : 17:18:02, Sep 19 2022
>
>
> U-Boot 2022.10-rc5-00003-g9e02955991-dirty (Sep 19 2022 - 21:49:01 -0300)
>
> CPU: Freescale i.MX8MMQ rev1.0 at 1200 MHz
> Reset cause: POR
> Model: FSL i.MX8MM EVK board
> DRAM: 2 GiB
> Core: 176 devices, 23 uclasses, devicetree: separate
> WDT: Started watchdog@30280000 with servicing (60s timeout)
> MMC: FSL_SDHC: 1, FSL_SDHC: 2
> Loading Environment from MMC... OK
> In: serial@30890000
> Out: serial@30890000
> Err: serial@30890000
> SEC0: RNG instantiated
> Net: eth0: ethernet@30be0000
> Hit any key to stop autoboot: 0
> u-boot=>
>
> One issue that I observed and haven't had a chance to debug yet is that
> running 'ums 0 mmc 1' works only when issued for the first time.
>
> If I interrupt with CRLC+C and then launch 'ums 0 mmc 1' again, it
> does not work.
>
> This is unrelated to SDP and happens when I boot from SD card as well.
>
> If you have any ideas, please let me know.

I am not sure, the master branch has this issue? Any specific config
option required?

Thanks,
Peng.

>
> Thanks,
>
> Fabio Estevam