From: Baruch Siach <baruch@tkos.co.il>
To: u-boot@lists.denx.de
Subject: [PATCH 10/10] arm: mvebu: clearfog: don't assume MMC booting
Date: Mon, 13 Jan 2020 13:40:23 +0200 [thread overview]
Message-ID: <878smbzis8.fsf@tarshish> (raw)
In-Reply-To: <e9291f20-6f93-ed67-fc68-c55b4ac1ffe6@denx.de>
Hi Stefan,
On Mon, Jan 13 2020, 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. And please do a compile run for all targets as
> well (Travis). I'll try to push it upstream in this merge window.
Thanks. I'll work on that tomorrow.
While you are at it can you also push this fix:
https://patchwork.ozlabs.org/patch/1205597/
Clearfog can't boot v2020.01 clearfog_defconfig without it :-(.
Thanks,
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
next prev parent reply other threads:[~2020-01-13 11:40 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-11 19:36 [PATCH 01/10] arm: mvebu: fix SerDes table alignment Joel Johnson
2020-01-11 19:36 ` [PATCH 02/10] arm: mvebu: solidrun: remove hardcoded DTS MAC address Joel Johnson
2020-01-13 8:21 ` Stefan Roese
2020-01-11 19:36 ` [PATCH 03/10] arm: mvebu: clearfog: initial ClearFog Base variant Joel Johnson
2020-01-12 10:14 ` Baruch Siach
2020-01-12 15:16 ` Joel Johnson
2020-01-13 8:19 ` Stefan Roese
2020-01-11 19:36 ` [PATCH 04/10] arm: mvebu: clearfog: Add SATA mode flags Joel Johnson
2020-01-11 19:36 ` [PATCH 05/10] arm: mvebu: clearfog: Add option for 2.5 Gbps SFP Joel Johnson
2020-01-12 10:21 ` Baruch Siach
2020-01-12 15:07 ` Joel Johnson
2020-01-11 19:36 ` [PATCH 06/10] arm: mvebu: clearfog: Add config for 2GB SOM Joel Johnson
2020-01-12 10:33 ` Baruch Siach
2020-01-12 15:48 ` Joel Johnson
2020-01-12 16:44 ` Baruch Siach
2020-01-11 19:36 ` [PATCH 07/10] arm: mvebu: clearfog: add SPI offsets Joel Johnson
2020-01-12 10:42 ` Baruch Siach
2020-01-12 15:30 ` Joel Johnson
2020-01-12 16:28 ` Baruch Siach
2020-01-11 19:36 ` [PATCH 08/10] arm: mvebu: enable working default boot support Joel Johnson
2020-01-11 21:07 ` Joel Johnson
2020-01-13 8:26 ` Stefan Roese
2020-01-11 19:36 ` [PATCH 09/10] arm: mvebu: clearfog: move ENV params to Kconfig Joel Johnson
2020-01-13 8:29 ` Stefan Roese
2020-01-11 19:36 ` [PATCH 10/10] arm: mvebu: clearfog: don't assume MMC booting Joel Johnson
2020-01-12 10:49 ` Baruch Siach
2020-01-12 15:40 ` Joel Johnson
2020-01-12 16:34 ` Baruch Siach
2020-01-13 6:48 ` Stefan Roese
2020-01-13 11:40 ` Baruch Siach [this message]
2020-01-13 11:42 ` Stefan Roese
2020-01-14 12:55 ` Baruch Siach
2020-01-14 13:01 ` Stefan Roese
2020-01-14 14:53 ` Baruch Siach
2020-01-14 15:06 ` Stefan Roese
2020-01-15 7:04 ` Baruch Siach
2020-01-13 8:18 ` [PATCH 01/10] arm: mvebu: fix SerDes table alignment Stefan Roese
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=878smbzis8.fsf@tarshish \
--to=baruch@tkos.co.il \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox