u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* Please test U-Boot Allwinner F1C100s support
@ 2022-01-31  2:04 Andre Przywara
  2022-01-31  2:12 ` Jesse Taube
  0 siblings, 1 reply; 11+ messages in thread
From: Andre Przywara @ 2022-01-31  2:04 UTC (permalink / raw)
  To: Jesse Taube, Samuel Holland, Giulio Benetti, George Hilliard,
	Chris Morgan
  Cc: U-Boot Mailing List, linux-sunxi

Hi,

thanks to the tireless work of Jesse, I think the basic Allwinner
F1C100s support for mainline U-Boot is now ready to be merged. Since I
don't have the hardware, can any of you who have boards please give it a
try on their hardware? I would really love to see some confirmations
before I merge it.
I put a branch here:
https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/tree/f1c100s

This should probably work on any F1C100s board, using the LicheePi Nano
defconfig. Also curious about the F1C200s (looking at you, George ;-)

This is very basic support at this point, there is no USB or SPI
support, and even the SD card is not usable from U-Boot proper. I am
hopeful we can add MMC and SPI support still in this cycle, but I don't
want to hold back this series any longer, and it should be relatively
simple support patches.

What should work though is booting via FEL and from SD card (the SPL
MMC code is in place, if I am not mistaken). Just booting it to a U-Boot
prompt via both methods would be a great test already. Maybe you can
even upload a kernel and initrd via FEL and boot it from there, using
$fdtcontroladdr for the DT address, for instance.

Any replies would be highly appreciated!

Many thanks,
Andre

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31  2:04 Please test U-Boot Allwinner F1C100s support Andre Przywara
@ 2022-01-31  2:12 ` Jesse Taube
  2022-01-31 10:47   ` Giulio Benetti
  0 siblings, 1 reply; 11+ messages in thread
From: Jesse Taube @ 2022-01-31  2:12 UTC (permalink / raw)
  To: Andre Przywara, Samuel Holland, Giulio Benetti, George Hilliard,
	Chris Morgan
  Cc: U-Boot Mailing List, linux-sunxi



On 1/30/22 21:04, Andre Przywara wrote:
> Hi,
> 
> thanks to the tireless work of Jesse, I think the basic Allwinner
> F1C100s support for mainline U-Boot is now ready to be merged. Since I
> don't have the hardware, can any of you who have boards please give it a
> try on their hardware? I would really love to see some confirmations
> before I merge it.

I really have to thank you for reviewing!

> I put a branch here:
> https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/tree/f1c100s
> 
> This should probably work on any F1C100s board, using the LicheePi Nano
> defconfig. Also curious about the F1C200s (looking at you, George ;-)

I have tested on both. Giulio has them aswell now.

> 
> This is very basic support at this point, there is no USB or SPI
> support, and even the SD card is not usable from U-Boot proper.

Sd card will work if we add it to the dts.

> I am hopeful we can add MMC and SPI support still in this cycle, but I don't
> want to hold back this series any longer, and it should be relatively
> simple support patches.

I have patches for both spi-boot and U-Boot proper spi suport.
I did manage to get it to find the boot device with:
`u32 brom_call = *(u32*)(fel_stash.sp-4);` in mach-sunxi/board.c with 
the suggestion that Stephen said. Should I submit after it gets into 
master or is tomorrow fine.

Many thanks,
	Jesse Taube
> 
> What should work though is booting via FEL and from SD card (the SPL
> MMC code is in place, if I am not mistaken). Just booting it to a U-Boot
> prompt via both methods would be a great test already. Maybe you can
> even upload a kernel and initrd via FEL and boot it from there, using
> $fdtcontroladdr for the DT address, for instance.
> 
> Any replies would be highly appreciated!
> 
> Many thanks,
> Andre

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31  2:12 ` Jesse Taube
@ 2022-01-31 10:47   ` Giulio Benetti
  2022-01-31 12:17     ` Giulio Benetti
  0 siblings, 1 reply; 11+ messages in thread
From: Giulio Benetti @ 2022-01-31 10:47 UTC (permalink / raw)
  To: Jesse Taube
  Cc: Andre Przywara, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

Hi Andre and Jesse,

I test it today on both F1C100s and F1C200s!
Lichee Pi Nano and the Jesse’s board with F1C200s.

Best regards
—-
Giulio Benetti
Benetti Engineering sas

> Il giorno 31 gen 2022, alle ore 03:12, Jesse Taube <mr.bossman075@gmail.com> ha scritto:
> 
> 
> 
>> On 1/30/22 21:04, Andre Przywara wrote:
>> Hi,
>> thanks to the tireless work of Jesse, I think the basic Allwinner
>> F1C100s support for mainline U-Boot is now ready to be merged. Since I
>> don't have the hardware, can any of you who have boards please give it a
>> try on their hardware? I would really love to see some confirmations
>> before I merge it.
> 
> I really have to thank you for reviewing!
> 
>> I put a branch here:
>> https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/tree/f1c100s
>> This should probably work on any F1C100s board, using the LicheePi Nano
>> defconfig. Also curious about the F1C200s (looking at you, George ;-)
> 
> I have tested on both. Giulio has them aswell now.
> 
>> This is very basic support at this point, there is no USB or SPI
>> support, and even the SD card is not usable from U-Boot proper.
> 
> Sd card will work if we add it to the dts.
> 
>> I am hopeful we can add MMC and SPI support still in this cycle, but I don't
>> want to hold back this series any longer, and it should be relatively
>> simple support patches.
> 
> I have patches for both spi-boot and U-Boot proper spi suport.
> I did manage to get it to find the boot device with:
> `u32 brom_call = *(u32*)(fel_stash.sp-4);` in mach-sunxi/board.c with the suggestion that Stephen said. Should I submit after it gets into master or is tomorrow fine.
> 
> Many thanks,
>    Jesse Taube
>> What should work though is booting via FEL and from SD card (the SPL
>> MMC code is in place, if I am not mistaken). Just booting it to a U-Boot
>> prompt via both methods would be a great test already. Maybe you can
>> even upload a kernel and initrd via FEL and boot it from there, using
>> $fdtcontroladdr for the DT address, for instance.
>> Any replies would be highly appreciated!
>> Many thanks,
>> Andre


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 10:47   ` Giulio Benetti
@ 2022-01-31 12:17     ` Giulio Benetti
  2022-01-31 12:57       ` Andre Przywara
  0 siblings, 1 reply; 11+ messages in thread
From: Giulio Benetti @ 2022-01-31 12:17 UTC (permalink / raw)
  To: Jesse Taube
  Cc: Andre Przywara, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

Hi All,

I've tried several times on both F1C100s and F1C200s, maybe I'm doing 
something wrong but it doesn't work :-/

I burn sd-card with:
# sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8

I've triple-checked that the file has been written there by dumping the 
sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI 
image for example. But no uart output.

Am I doing something wrong?

Best regards
-- 
Giulio Benetti
Benetti Engineering sas

On 31/01/22 11:47, Giulio Benetti wrote:
> Hi Andre and Jesse,
> 
> I test it today on both F1C100s and F1C200s!
> Lichee Pi Nano and the Jesse’s board with F1C200s.
> 
> Best regards
> —-
> Giulio Benetti
> Benetti Engineering sas
> 
>> Il giorno 31 gen 2022, alle ore 03:12, Jesse Taube <mr.bossman075@gmail.com> ha scritto:
>>
>> 
>>
>>> On 1/30/22 21:04, Andre Przywara wrote:
>>> Hi,
>>> thanks to the tireless work of Jesse, I think the basic Allwinner
>>> F1C100s support for mainline U-Boot is now ready to be merged. Since I
>>> don't have the hardware, can any of you who have boards please give it a
>>> try on their hardware? I would really love to see some confirmations
>>> before I merge it.
>>
>> I really have to thank you for reviewing!
>>
>>> I put a branch here:
>>> https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/tree/f1c100s
>>> This should probably work on any F1C100s board, using the LicheePi Nano
>>> defconfig. Also curious about the F1C200s (looking at you, George ;-)
>>
>> I have tested on both. Giulio has them aswell now.
>>
>>> This is very basic support at this point, there is no USB or SPI
>>> support, and even the SD card is not usable from U-Boot proper.
>>
>> Sd card will work if we add it to the dts.
>>
>>> I am hopeful we can add MMC and SPI support still in this cycle, but I don't
>>> want to hold back this series any longer, and it should be relatively
>>> simple support patches.
>>
>> I have patches for both spi-boot and U-Boot proper spi suport.
>> I did manage to get it to find the boot device with:
>> `u32 brom_call = *(u32*)(fel_stash.sp-4);` in mach-sunxi/board.c with the suggestion that Stephen said. Should I submit after it gets into master or is tomorrow fine.
>>
>> Many thanks,
>>     Jesse Taube
>>> What should work though is booting via FEL and from SD card (the SPL
>>> MMC code is in place, if I am not mistaken). Just booting it to a U-Boot
>>> prompt via both methods would be a great test already. Maybe you can
>>> even upload a kernel and initrd via FEL and boot it from there, using
>>> $fdtcontroladdr for the DT address, for instance.
>>> Any replies would be highly appreciated!
>>> Many thanks,
>>> Andre
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 12:17     ` Giulio Benetti
@ 2022-01-31 12:57       ` Andre Przywara
  2022-01-31 13:07         ` Giulio Benetti
  0 siblings, 1 reply; 11+ messages in thread
From: Andre Przywara @ 2022-01-31 12:57 UTC (permalink / raw)
  To: Giulio Benetti
  Cc: Jesse Taube, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

On Mon, 31 Jan 2022 13:17:52 +0100
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

> Hi All,
> 
> I've tried several times on both F1C100s and F1C200s, maybe I'm doing 
> something wrong but it doesn't work :-/
> 
> I burn sd-card with:
> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
> 
> I've triple-checked that the file has been written there by dumping the 
> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI 
> image for example. But no uart output.

I haven't looked in detail what would be needed for the SPL part of the
MMC driver to really work, I just checked that the base addresses are
correct, the pinmux looked alright as well.
The DT nodes are not needed for the SPL, and I don't see other changes
related to MMC in the original series?
Do you see UART output from the SPL, or nothing at all?
And you are not using UART0 on the PortF pins, so multiplexed with the
MMC, I guess?

Cheers,
Andre



> 
> Am I doing something wrong?
> 
> Best regards


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 12:57       ` Andre Przywara
@ 2022-01-31 13:07         ` Giulio Benetti
  2022-01-31 13:57           ` Andre Przywara
  2022-01-31 18:13           ` Jesse Taube
  0 siblings, 2 replies; 11+ messages in thread
From: Giulio Benetti @ 2022-01-31 13:07 UTC (permalink / raw)
  To: Andre Przywara
  Cc: Jesse Taube, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

Hi Andre,

> Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
> 
> On Mon, 31 Jan 2022 13:17:52 +0100
> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> 
>> Hi All,
>> 
>> I've tried several times on both F1C100s and F1C200s, maybe I'm doing 
>> something wrong but it doesn't work :-/
>> 
>> I burn sd-card with:
>> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
>> 
>> I've triple-checked that the file has been written there by dumping the 
>> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI 
>> image for example. But no uart output.
> 
> I haven't looked in detail what would be needed for the SPL part of the
> MMC driver to really work, I just checked that the base addresses are
> correct, the pinmux looked alright as well.
> The DT nodes are not needed for the SPL, and I don't see other changes
> related to MMC in the original series?

Yes that’s true.

> Do you see UART output from the SPL, or nothing at all?

Unfortunately nothing either with SPL

> And you are not using UART0 on the PortF pins, so multiplexed with the
> MMC, I guess?

I use PE0 and PE1 Uart0 according to dts.

Have you got such board to test it?
Or we can wait for Jesse or George.

I’ve used the git you’ve pointed.

Anyway I think we’re very close to have it working.

At least my work “behind the scenes” will be worth, right Jesse :-)?

Best regards
—-
Giulio Benetti
Benetti Engineering sas

> 
> Cheers,
> Andre
> 
> 
> 
>> 
>> Am I doing something wrong?
>> 
>> Best regards
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 13:07         ` Giulio Benetti
@ 2022-01-31 13:57           ` Andre Przywara
  2022-01-31 14:12             ` Giulio Benetti
  2022-01-31 18:13           ` Jesse Taube
  1 sibling, 1 reply; 11+ messages in thread
From: Andre Przywara @ 2022-01-31 13:57 UTC (permalink / raw)
  To: Giulio Benetti
  Cc: Jesse Taube, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

On Mon, 31 Jan 2022 14:07:54 +0100
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

Hi Giulio,

> > Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
> > 
> > On Mon, 31 Jan 2022 13:17:52 +0100
> > Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> >   
> >> Hi All,
> >> 
> >> I've tried several times on both F1C100s and F1C200s, maybe I'm doing 
> >> something wrong but it doesn't work :-/
> >> 
> >> I burn sd-card with:
> >> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
> >> 
> >> I've triple-checked that the file has been written there by dumping the 
> >> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI 
> >> image for example. But no uart output.  
> > 
> > I haven't looked in detail what would be needed for the SPL part of the
> > MMC driver to really work, I just checked that the base addresses are
> > correct, the pinmux looked alright as well.
> > The DT nodes are not needed for the SPL, and I don't see other changes
> > related to MMC in the original series?  
> 
> Yes that’s true.
> 
> > Do you see UART output from the SPL, or nothing at all?  
> 
> Unfortunately nothing either with SPL

That is interesting. If there would be a problem with the MMC code, you
should at least see the SPL banner, and then see it failing loading U-Boot
proper. I take it FEL boot works? Maybe we miss something with the clock
setup that the BROM does for us during FEL operation?

Can you try to clear the area between the end of the SPL and the beginning
of the U-Boot proper image (dd if=/dev/zero of=/dev/sdX bs=1k seek=32
count=8)?
Does any other branch work with SD card booting? Can you check the SPL
size in there (with hexdump -C u-boot-sunxi-with-spl.bin | head, check for
the halfword at 0x10)?

> > And you are not using UART0 on the PortF pins, so multiplexed with the
> > MMC, I guess?  
> 
> I use PE0 and PE1 Uart0 according to dts.
> 
> Have you got such board to test it?

That's the problem: I am driving blindfolded, because I don't have
hardware :-(

Cheers,
Andre

> Or we can wait for Jesse or George.
> 
> I’ve used the git you’ve pointed.
> 
> Anyway I think we’re very close to have it working.
> 
> At least my work “behind the scenes” will be worth, right Jesse :-)?
> 
> Best regards
> —-
> Giulio Benetti
> Benetti Engineering sas
> 
> > 
> > Cheers,
> > Andre
> > 
> > 
> >   
> >> 
> >> Am I doing something wrong?
> >> 
> >> Best regards  
> >   
> 
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 13:57           ` Andre Przywara
@ 2022-01-31 14:12             ` Giulio Benetti
  2022-01-31 14:25               ` Andre Przywara
  0 siblings, 1 reply; 11+ messages in thread
From: Giulio Benetti @ 2022-01-31 14:12 UTC (permalink / raw)
  To: Andre Przywara
  Cc: Jesse Taube, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

Hi Andre,

On 31/01/22 14:57, Andre Przywara wrote:
> On Mon, 31 Jan 2022 14:07:54 +0100
> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> 
> Hi Giulio,
> 
>>> Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
>>>
>>> On Mon, 31 Jan 2022 13:17:52 +0100
>>> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
>>>    
>>>> Hi All,
>>>>
>>>> I've tried several times on both F1C100s and F1C200s, maybe I'm doing
>>>> something wrong but it doesn't work :-/
>>>>
>>>> I burn sd-card with:
>>>> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
>>>>
>>>> I've triple-checked that the file has been written there by dumping the
>>>> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI
>>>> image for example. But no uart output.
>>>
>>> I haven't looked in detail what would be needed for the SPL part of the
>>> MMC driver to really work, I just checked that the base addresses are
>>> correct, the pinmux looked alright as well.
>>> The DT nodes are not needed for the SPL, and I don't see other changes
>>> related to MMC in the original series?
>>
>> Yes that’s true.
>>
>>> Do you see UART output from the SPL, or nothing at all?
>>
>> Unfortunately nothing either with SPL
> 
> That is interesting. If there would be a problem with the MMC code, you
> should at least see the SPL banner, and then see it failing loading U-Boot
> proper. 

Yes

> I take it FEL boot works? 

Yes, it does work on both F1C100s and 200s boards :-/

> Maybe we miss something with the clock
> setup that the BROM does for us during FEL operation?

Yes, it sounds like that, it's a classic

> Can you try to clear the area between the end of the SPL and the beginning
> of the U-Boot proper image (dd if=/dev/zero of=/dev/sdX bs=1k seek=32
> count=8)?

I've zeroed the first 32MB of sd-card now, so we're sure nothing can 
interfere.

# sudo dd if=/dev/zero of=/dev/sda bs=1M count=32
# sync
# sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sda bs=1k seek=8
# sync

(/dev/sda is my sd-card for real since I have m.2 ssd)

> Does any other branch work with SD card booting? Can you check the SPL
> size in there (with hexdump -C u-boot-sunxi-with-spl.bin | head, check for
> the halfword at 0x10)?

This is the result of the command:
```
00000000  16 00 00 ea 65 47 4f 4e  2e 42 54 30 64 04 71 7c 
|....eGON.BT0d.q||
00000010  00 60 00 00 53 50 4c 02  00 00 00 00 00 00 00 00 
|.`..SPL.........|
00000020  2c 00 00 00 00 00 00 00  00 00 00 00 73 75 6e 69 
|,...........suni|
00000030  76 2d 66 31 63 31 30 30  73 2d 6c 69 63 68 65 65 
|v-f1c100s-lichee|
00000040  70 69 2d 6e 61 6e 6f 00  00 00 00 00 00 00 00 00 
|pi-nano.........|
00000050  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00 
|................|
00000060  0f 00 00 ea 14 f0 9f e5  14 f0 9f e5 14 f0 9f e5 
|................|
00000070  14 f0 9f e5 14 f0 9f e5  14 f0 9f e5 14 f0 9f e5 
|................|
00000080  a0 00 00 00 a0 00 00 00  a0 00 00 00 a0 00 00 00 
|................|
00000090  a0 00 00 00 a0 00 00 00  a0 00 00 00 ef be ad de 
|................|
```

>>> And you are not using UART0 on the PortF pins, so multiplexed with the
>>> MMC, I guess?
>>
>> I use PE0 and PE1 Uart0 according to dts.
>>
>> Have you got such board to test it?
> 
> That's the problem: I am driving blindfolded, because I don't have
> hardware :-(

Yeah, but I've read you've ordered it! :-)

Jesse has it working so maybe he can check how clocks are setup after 
BROM using u-boot itself. I've never understood why Allwinner/Rockchip 
place jtag on the sd-card pins. I really hate it, it would take so few 
with free jtag pins to check this :-/

Ciao :-)
-- 
Giulio Benetti
Benetti Engineering sas

> Cheers,
> Andre
> 
>> Or we can wait for Jesse or George.
>>
>> I’ve used the git you’ve pointed.
>>
>> Anyway I think we’re very close to have it working.
>>
>> At least my work “behind the scenes” will be worth, right Jesse :-)?
>>
>> Best regards
>> —-
>> Giulio Benetti
>> Benetti Engineering sas
>>
>>>
>>> Cheers,
>>> Andre
>>>
>>>
>>>    
>>>>
>>>> Am I doing something wrong?
>>>>
>>>> Best regards
>>>    
>>
>>
> 


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 14:12             ` Giulio Benetti
@ 2022-01-31 14:25               ` Andre Przywara
  0 siblings, 0 replies; 11+ messages in thread
From: Andre Przywara @ 2022-01-31 14:25 UTC (permalink / raw)
  To: Giulio Benetti
  Cc: Jesse Taube, Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

On Mon, 31 Jan 2022 15:12:23 +0100
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

> Hi Andre,
> 
> On 31/01/22 14:57, Andre Przywara wrote:
> > On Mon, 31 Jan 2022 14:07:54 +0100
> > Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> > 
> > Hi Giulio,
> >   
> >>> Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
> >>>
> >>> On Mon, 31 Jan 2022 13:17:52 +0100
> >>> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> >>>      
> >>>> Hi All,
> >>>>
> >>>> I've tried several times on both F1C100s and F1C200s, maybe I'm doing
> >>>> something wrong but it doesn't work :-/
> >>>>
> >>>> I burn sd-card with:
> >>>> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
> >>>>
> >>>> I've triple-checked that the file has been written there by dumping the
> >>>> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI
> >>>> image for example. But no uart output.  
> >>>
> >>> I haven't looked in detail what would be needed for the SPL part of the
> >>> MMC driver to really work, I just checked that the base addresses are
> >>> correct, the pinmux looked alright as well.
> >>> The DT nodes are not needed for the SPL, and I don't see other changes
> >>> related to MMC in the original series?  
> >>
> >> Yes that’s true.
> >>  
> >>> Do you see UART output from the SPL, or nothing at all?  
> >>
> >> Unfortunately nothing either with SPL  
> > 
> > That is interesting. If there would be a problem with the MMC code, you
> > should at least see the SPL banner, and then see it failing loading U-Boot
> > proper.   
> 
> Yes
> 
> > I take it FEL boot works?   
> 
> Yes, it does work on both F1C100s and 200s boards :-/
> 
> > Maybe we miss something with the clock
> > setup that the BROM does for us during FEL operation?  
> 
> Yes, it sounds like that, it's a classic
> 
> > Can you try to clear the area between the end of the SPL and the beginning
> > of the U-Boot proper image (dd if=/dev/zero of=/dev/sdX bs=1k seek=32
> > count=8)?  
> 
> I've zeroed the first 32MB of sd-card now, so we're sure nothing can 
> interfere.
> 
> # sudo dd if=/dev/zero of=/dev/sda bs=1M count=32
> # sync
> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sda bs=1k seek=8
> # sync
> 
> (/dev/sda is my sd-card for real since I have m.2 ssd)

Sorry, I meant to clear those bits after writing to the card (or clear that
in the image file). binman now manages the packaging, so places 0xff into
the 8K gap between our 24K SPL and the 32KB offset we place U-Boot proper
at. If the BROM would read beyond the 24KB, the checksum would fail,
but zeros are ignored by the algorithm, so we are a bit more robust. Also
it might be interesting to set the size of 32KB (fixing up the checksum).
What I was after is that no output could also mean that the BROM dismissed
the SPL it found on the SD card. Do you see FEL mode enabled afterwards?

> > Does any other branch work with SD card booting? Can you check the SPL
> > size in there (with hexdump -C u-boot-sunxi-with-spl.bin | head, check for
> > the halfword at 0x10)?  
> 
> This is the result of the command:
> ```
> 00000000  16 00 00 ea 65 47 4f 4e  2e 42 54 30 64 04 71 7c 
> |....eGON.BT0d.q||
> 00000010  00 60 00 00 53 50 4c 02  00 00 00 00 00 00 00 00 
> |.`..SPL.........|
> 00000020  2c 00 00 00 00 00 00 00  00 00 00 00 73 75 6e 69 
> |,...........suni|
> 00000030  76 2d 66 31 63 31 30 30  73 2d 6c 69 63 68 65 65 
> |v-f1c100s-lichee|
> 00000040  70 69 2d 6e 61 6e 6f 00  00 00 00 00 00 00 00 00 
> |pi-nano.........|
> 00000050  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00 
> |................|
> 00000060  0f 00 00 ea 14 f0 9f e5  14 f0 9f e5 14 f0 9f e5 
> |................|
> 00000070  14 f0 9f e5 14 f0 9f e5  14 f0 9f e5 14 f0 9f e5 
> |................|
> 00000080  a0 00 00 00 a0 00 00 00  a0 00 00 00 a0 00 00 00 
> |................|
> 00000090  a0 00 00 00 a0 00 00 00  a0 00 00 00 ef be ad de 
> |................|
> ```
> 
> >>> And you are not using UART0 on the PortF pins, so multiplexed with the
> >>> MMC, I guess?  
> >>
> >> I use PE0 and PE1 Uart0 according to dts.
> >>
> >> Have you got such board to test it?  
> > 
> > That's the problem: I am driving blindfolded, because I don't have
> > hardware :-(  
> 
> Yeah, but I've read you've ordered it! :-)

Indeed, but it ships from Hongkong. Even without Chinese New Year that
would probably take another week to arrive.

Cheers,
Andre

> Jesse has it working so maybe he can check how clocks are setup after 
> BROM using u-boot itself. I've never understood why Allwinner/Rockchip 
> place jtag on the sd-card pins. I really hate it, it would take so few 
> with free jtag pins to check this :-/
> 
> Ciao :-)


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 13:07         ` Giulio Benetti
  2022-01-31 13:57           ` Andre Przywara
@ 2022-01-31 18:13           ` Jesse Taube
  2022-01-31 20:44             ` Giulio Benetti
  1 sibling, 1 reply; 11+ messages in thread
From: Jesse Taube @ 2022-01-31 18:13 UTC (permalink / raw)
  To: Giulio Benetti, Andre Przywara
  Cc: Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi



On 1/31/22 08:07, Giulio Benetti wrote:
> Hi Andre,
> 
>> Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
>>
>> On Mon, 31 Jan 2022 13:17:52 +0100
>> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
>>
>>> Hi All,
>>>
>>> I've tried several times on both F1C100s and F1C200s, maybe I'm doing
>>> something wrong but it doesn't work :-/
>>>
>>> I burn sd-card with:
>>> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
>>>
>>> I've triple-checked that the file has been written there by dumping the
>>> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI
>>> image for example. But no uart output.
>>
>> I haven't looked in detail what would be needed for the SPL part of the
>> MMC driver to really work, I just checked that the base addresses are
>> correct, the pinmux looked alright as well.
>> The DT nodes are not needed for the SPL, and I don't see other changes
>> related to MMC in the original series?
> 
> Yes that’s true.
> 
>> Do you see UART output from the SPL, or nothing at all?
> 
> Unfortunately nothing either with SPL
> 
>> And you are not using UART0 on the PortF pins, so multiplexed with the
>> MMC, I guess?
> 
> I use PE0 and PE1 Uart0 according to dts.
> 
> Have you got such board to test it?
> Or we can wait for Jesse or George.
> 
> I’ve used the git you’ve pointed.
> 
> Anyway I think we’re very close to have it working.
> 
> At least my work “behind the scenes” will be worth, right Jesse :-)?
Yes, I'm sorry there wasn't much mention of you on this patch.
I Just woke up, sorry i couldn't be here earlier, I'll take a look with 
Giulio.
> 
> Best regards
> —-
> Giulio Benetti
> Benetti Engineering sas
> 
>>
>> Cheers,
>> Andre
>>
>>
>>
>>>
>>> Am I doing something wrong?
>>>
>>> Best regards
>>
> 

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Please test U-Boot Allwinner F1C100s support
  2022-01-31 18:13           ` Jesse Taube
@ 2022-01-31 20:44             ` Giulio Benetti
  0 siblings, 0 replies; 11+ messages in thread
From: Giulio Benetti @ 2022-01-31 20:44 UTC (permalink / raw)
  To: Jesse Taube, Andre Przywara
  Cc: Samuel Holland, George Hilliard, Chris Morgan,
	U-Boot Mailing List, linux-sunxi

Hi Andre, Jesse, All,

On 31/01/22 19:13, Jesse Taube wrote:
> 
> 
> On 1/31/22 08:07, Giulio Benetti wrote:
>> Hi Andre,
>>
>>> Il giorno 31 gen 2022, alle ore 13:57, Andre Przywara <andre.przywara@foss.arm.com> ha scritto:
>>>
>>> On Mon, 31 Jan 2022 13:17:52 +0100
>>> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
>>>
>>>> Hi All,
>>>>
>>>> I've tried several times on both F1C100s and F1C200s, maybe I'm doing
>>>> something wrong but it doesn't work :-/
>>>>
>>>> I burn sd-card with:
>>>> # sudo dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1k seek=8
>>>>
>>>> I've triple-checked that the file has been written there by dumping the
>>>> sd-card and I see it's written at 0x2000, so 8K offset, like the mangoPI
>>>> image for example. But no uart output.
>>>
>>> I haven't looked in detail what would be needed for the SPL part of the
>>> MMC driver to really work, I just checked that the base addresses are
>>> correct, the pinmux looked alright as well.
>>> The DT nodes are not needed for the SPL, and I don't see other changes
>>> related to MMC in the original series?
>>
>> Yes that’s true.
>>
>>> Do you see UART output from the SPL, or nothing at all?
>>
>> Unfortunately nothing either with SPL
>>
>>> And you are not using UART0 on the PortF pins, so multiplexed with the
>>> MMC, I guess?
>>
>> I use PE0 and PE1 Uart0 according to dts.
>>
>> Have you got such board to test it?
>> Or we can wait for Jesse or George.
>>
>> I’ve used the git you’ve pointed.
>>
>> Anyway I think we’re very close to have it working.
>>
>> At least my work “behind the scenes” will be worth, right Jesse :-)?
> Yes, I'm sorry there wasn't much mention of you on this patch.
> I Just woke up, sorry i couldn't be here earlier, I'll take a look with
> Giulio.

I've tested succesfully on both F1C100s and 200s boards!
https://source.denx.de/u-boot/custodians/u-boot-sunxi/-/tree/f1c100s

I've realized with Jesse that here with me I have 3 USB->Serial 
converter broken!!! With the 4th it worked and I've checked with scope 
that was hidden by garbage on my desk :-)

So it works! Sorry for the noise

Kind regards
-- 
Giulio Benetti
Benetti Engineering sas

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2022-01-31 20:44 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-31  2:04 Please test U-Boot Allwinner F1C100s support Andre Przywara
2022-01-31  2:12 ` Jesse Taube
2022-01-31 10:47   ` Giulio Benetti
2022-01-31 12:17     ` Giulio Benetti
2022-01-31 12:57       ` Andre Przywara
2022-01-31 13:07         ` Giulio Benetti
2022-01-31 13:57           ` Andre Przywara
2022-01-31 14:12             ` Giulio Benetti
2022-01-31 14:25               ` Andre Przywara
2022-01-31 18:13           ` Jesse Taube
2022-01-31 20:44             ` Giulio Benetti

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).