From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Mon, 12 Feb 2018 09:48:13 +0100 Subject: [U-Boot] [PATCH 8/8] Migrate generic bootcount to Kconfig In-Reply-To: References: <1518350813-3418-1-git-send-email-alex.kiernan@gmail.com> <1518350813-3418-9-git-send-email-alex.kiernan@gmail.com> <20180211203646.24102b36@jawa> <20180211224440.21778e77@jawa> Message-ID: <20180212094813.0377f521@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Alex, > On Sun, Feb 11, 2018 at 9:44 PM, Lukasz Majewski > wrote: > > On Sun, 11 Feb 2018 21:04:46 +0000 > > Alex Kiernan wrote: > > > >> > >> That said, squashing in that change doesn't obviously break > >> anything for me, and is probably a step in the right direction. > >> > >> I'll see what Travis thinks. > >> > > > > We will probably receive build breaks... > > Yup... https://travis-ci.org/akiernan/u-boot/jobs/340344489 > > Just tried again on one of those failures (x600) with the the default > removed and just set the on board that uses CONFIG_EXT, but that then > fails at config time. Ok. I see > > TBH I'd actually like to take it out of Kconfig (which I realise is > the wrong direction) as it just feels fundamentally wrong the way it > is. To finish moving SYS_BOOTCOUNT_ADDR to Kconfig we would need to add its definition to each eligible ./configs/_defconfig file. Then we would have the previous behaviour preserved. > I don't know what the U-Boot approach configuration like this > is... struggling to find prior art. Let's ask Tom for his opinion as he did much such work before. > 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-10 Fax: (+49)-8142-66989-80 Email: wd 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: