Hi Baruch,

On 14.01.20 13:55, Baruch Siach wrote:
Hi Stefan,

On Mon, Jan 13, 2020 at 07:48:34AM +0100, Stefan Roese wrote:
On 12.01.20 17:34, Baruch Siach wrote:
On Sun, Jan 12 2020, Joel Johnson wrote:
On 2020-01-12 03:49, Baruch Siach wrote:
On Sat, Jan 11 2020, Joel Johnson wrote:
+config SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
+       hex "Address on the MMC to load U-Boot from"
+       default 0x141
+       depends on MVEBU_SPL_BOOT_DEVICE_MMC

You might want to take a look at the series linked below, which
conflicts with this patch:

    https://patchwork.ozlabs.org/project/uboot/list/?series=121894

Indeed, the dynamic runtime approach seems much nicer for handling the MMCSD
offset business, including relative vs. absolute device offset. I only have
non-eMMC units so can't test it, but I'm all for dropping the
SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR portion of this patch in favor of that patch,
in which leaving the defconfig entry does no harm. Any indication of when that
patch will be included? It seems to have been stagnant for a while.

Indeed. I'm waiting patiently.

I have missed addressing this patch series for quite some time, sorry.
Could you please rebase this patchset on top of current mainline as it
doesn't apply any more.

The original patch series applies cleanly on top of current master as of
commit 88366b96ee0d86 ("travis: Switch to QEMU v4.2.0 globally"), and runs
successfully. It only trivially conflicts with the clearfog_defconfig
SPL_DM_GPIO fix (http://patchwork.ozlabs.org/patch/1205597/). Would you like
me to rebase on to of that?

Yes, please.
And please do a compile run for all targets as well (Travis). I'll try to
push it upstream in this merge window.

buildman for arm and aarch64 is currently running.

All platforms please. I suggest to use Travis for this. Be prepared that
this takes a few hours though.

Thanks,
Stefan

Reply via email to