On 20.08.20 14:03, Pali Rohár wrote:
On Thursday 20 August 2020 10:51:28 Pali Rohár wrote:
On Thursday 20 August 2020 10:17:55 Stefan Roese wrote:
On 20.08.20 09:40, Pali Rohár wrote:
Anyway, updating DTS has advantage that it is not needed to update
existing boot scripts for OS. There are more distributions for
Espressobin which have own boot scripts stored on SD card for loading
kernel. And therefore to use command line parameters it would be needed
to update all of them.
I see. This is an argument I understand. But can't you use the common
fdt_fixup_mtdparts() then?
And I see there another problem. For specifying size of mtd partitions
in command line, it is required to know offsets of those partitions. And
e.g. uboot env partition depends on CONFIG_ENV_OFFSET option which is
not available for uboot boot script code.
But if you have other idea, I'm open to also other solutions.
I have not investigated a multi-distribution solution here. Perhaps
the common fdt_fixup_mtdparts() is able to handle this?
In case U-Boot would see MTD SPI partitions, then fdt_fixup_mtdparts()
should work.
It looks like that fdt_fixup_mtdparts() is broken.
I added following config lines
CONFIG_MTDIDS_DEFAULT="nor0=nor0"
CONFIG_MTDPARTS_DEFAULT="nor0:4032k(firmware),64k(u-boot-env)"
CONFIG_FDT_FIXUP_PARTITIONS=y
Then in uboot console I called 'sf probe' and 'boot'.
And Linux kernel thrown following MTD error:
[ 1.038352] spi-nor spi0.0: w25q32dw (4096 Kbytes)
[ 1.043403] spi0.0: error parsing ofpart partition
/soc/internal-regs@d0000000/spi@10600/flash@0/partition@0
(/soc/internal-regs@d0000000/spi@10600/flash@0)
I know that passing mtdparts= is working in general. You need the
correct device names though. Perhaps something is wrong here. Hard
to say, without looking deeper. Could you please post the complete
Linux bootlog of this failing boot attempt?
So I do not know if it makes sense to continue debugging SF <--> MTD
layer in Uboot when passing uboot MTD partitions via
fdt_fixup_mtdparts() to Linux kernel does not work...
I understand. You have a working version and it makes not much sense
for you to work on an alternative solution. But the fdt_fixup_mtdparts()
way provides a common way for all (most?) boards and your's needs
to be rewritten for each board. So from this prospective, it definitely
makes sense to continue debugging / testing.
Original patch which in this thread is working fine and kernel correctly
see both defined partitions in ft_board_setup() function.
I understand. Thanks for at looking into this anyways.
Thanks,
Stefan
