From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <ptxdist-bounces@pengutronix.de>
MIME-Version: 1.0
References: <20191113170559.15307-1-bruno.thomsen@gmail.com>
 <20191115135935.GC1287@pengutronix.de>
 <f14b33d92fe457e6d5749b90a2bc621b0f6ff149.camel@diehl.com>
In-Reply-To: <f14b33d92fe457e6d5749b90a2bc621b0f6ff149.camel@diehl.com>
From: Bruno Thomsen <bruno.thomsen@gmail.com>
Date: Mon, 18 Nov 2019 13:25:39 +0100
Message-ID: <CAH+2xPBKMZsWL33EFk1u2s=qee6eE90tiPVFAF_dsZ54vR_NAA@mail.gmail.com>
Subject: Re: [ptxdist] [PATCH v2] u-boot: generate environment image
List-Id: PTXdist Development Mailing List <ptxdist.pengutronix.de>
List-Unsubscribe: <http://metis.pengutronix.de/cgi-bin/mailman/options/ptxdist>, 
 <mailto:ptxdist-request@pengutronix.de?subject=unsubscribe>
List-Archive: <http://metis.pengutronix.de/cgi-bin/mailman/private/ptxdist/>
List-Post: <mailto:ptxdist@pengutronix.de>
List-Help: <mailto:ptxdist-request@pengutronix.de?subject=help>
List-Subscribe: <http://metis.pengutronix.de/cgi-bin/mailman/listinfo/ptxdist>, 
 <mailto:ptxdist-request@pengutronix.de?subject=subscribe>
Reply-To: ptxdist@pengutronix.de
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ptxdist-bounces@pengutronix.de
Sender: "ptxdist" <ptxdist-bounces@pengutronix.de>
To: Denis OSTERLAND <denis.osterland@diehl.com>
Cc: "ada@thorsis.com" <ada@thorsis.com>, "m.olbrich@pengutronix.de" <m.olbrich@pengutronix.de>, "ptxdist@pengutronix.de" <ptxdist@pengutronix.de>, "bth@kamstrup.com" <bth@kamstrup.com>

Hi

Den man. 18. nov. 2019 kl. 09.43 skrev Denis OSTERLAND
<denis.osterland@diehl.com>:
>
> Hi,
>
> Am Freitag, den 15.11.2019, 14:59 +0100 schrieb Michael Olbrich:
> > On Wed, Nov 13, 2019 at 06:05:59PM +0100, Bruno Thomsen wrote:
> ...
> > > +config U_BOOT_DEFAULT_ENV_IMAGE_FILE
> > > +   prompt "Default environment image filename"
> > > +   string
> > > +   default "u-boot-env.img"
> > > +   help
> > > +     Change default environment output image name.
> >
> > Why the custom name? It's just the filename in images/ and it can be
> > changed wenn it's added to an image, if that's necessary.
> > If it's important, then the option should depend on
> > U_BOOT_DEFAULT_ENV_IMAGE

So it's easier to tell what environment image you flashing.

We use the image directly from images/ and does not create a
combined device image, but will remove both image names in
next version.

> > Denis, I think you were asking for a selectable input file, not generating
> > multiple env images?
> correct.
> I see no reason to have both files, or to specify the output file name.
> My suggestion was to give a filename or generate it via get_default_envs.sh if none given.
> If you want to be more explicit, I would prefer a choice between default and custom,
> where the filename depends on the custom selected.

I just overdid the function and will revert it to a simple choice between
none, default and custom.

> > And the same issue as above, if this stays: do we need to custom output
> > filename and hide the suboptions with 'depends'.

Will hide custom source file when choice is none or default.

/Bruno

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de