From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Tue, 21 May 2019 11:22:47 +0200 Subject: [U-Boot] [PATCH v2 2/2] cmd: Add 'bcb' command to read/modify/write BCB fields In-Reply-To: <20190521091352.GA5680@vmlxhi-102.adit-jv.com> References: <20190517144503.3275-1-erosca@de.adit-jv.com> <20190517144503.3275-3-erosca@de.adit-jv.com> <20190520072257.GA4084@vmlxhi-102.adit-jv.com> <20190521103659.0a0e084a@jawa> <20190521091352.GA5680@vmlxhi-102.adit-jv.com> Message-ID: <20190521112247.2be00787@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, 21 May 2019 11:13:52 +0200 Eugeniu Rosca wrote: > Hi Lukasz, > > On Tue, May 21, 2019 at 10:02:34AM +0100, Alex Kiernan wrote: > > On Tue, May 21, 2019 at 9:37 AM Lukasz Majewski > > wrote: > > > > > > Hi Alex, > > > > > > > On Mon, May 20, 2019 at 8:23 AM Eugeniu Rosca > > > > wrote: > > > > > > > > > > > > > > > > > Should it default to enabled if avb is used? > > > > > > > > > > I think at this specific moment in time, 'bcb' is orthogonal > > > > > (meaning it is neither a direct, nor a reverse dependency) to > > > > > any other Android feature in U-Boot. This could be > > > > > re-assessed, if platform maintainers start to rely on 'bcb' > > > > > in their U-Boot environments on regular basis. > > > > > > > > 'bcb' looks like something I'd be interested in, not running > > > > Android at all... currently I (ab)use the bootcounter to > > > > communicate between the kernel and U-Boot when I want to force > > > > a board through recovery, > > > > > > I don't know your exact use case, but wouldn't it be better to > > > use envs (with redundancy) and fw_setenv / fw_printenv from Linux > > > user space? > > > > > > Now those envs even support setting default values for u-boot (as > > > there is now separate library used for it). Moreover there is > > > OE/Yocto's recipe 'u-boot-fw-utils' which can be easily used and > > > installed. > > That's a truly constructive suggestion. Nevertheless, I believe this > would not work in case of CONFIG_ENV_IS_NOWHERE=y, which is how U-Boot > is built and used by the developers in our organization. I don't mind to see Android's "bcd" supported in U-Boot (I'm even happy for it). And yes - the CONFIG_ENV_IS_NOWHERE means that one loads the default envs (created at build time) to RAM for booting. Just one note (maybe you will find it useful) - it is possible to specify the default envs from external file: https://lists.denx.de/pipermail/u-boot/2018-March/323347.html As we don't have memory to store envs we cannot adjust or pass data via it. > > > > > It's a long story... I'm constrained by historic choices, which > > makes using the environment problematic. But you're right. > > > > -- > > Alex Kiernan > Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: