From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 14 Jan 2020 14:01:20 +0100 Subject: [PATCH 10/10] arm: mvebu: clearfog: don't assume MMC booting In-Reply-To: <20200114125522.7zab3rk3s6vofbde@sapphire.tkos.co.il> References: <20200111193639.19022-1-mrjoel@lixil.net> <20200111193639.19022-10-mrjoel@lixil.net> <87lfqdymnv.fsf@tarshish> <4524882422e2f769b54f3c4b6cd847ad@lixil.net> <87blr8zl8w.fsf@tarshish> <20200114125522.7zab3rk3s6vofbde@sapphire.tkos.co.il> Message-ID: <39437042-2b24-3979-1af4-25ccbde034e9@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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