From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig
Date: Fri, 18 Sep 2020 12:21:56 -0400 [thread overview]
Message-ID: <20200918162156.GV14816@bill-the-cat> (raw)
In-Reply-To: <d9ebaf89-aa5e-f242-a6fe-50aa745d7422@web.de>
On Fri, Sep 18, 2020 at 05:56:03PM +0200, Soeren Moch wrote:
>
>
> On 18.09.20 17:51, Stefano Babic wrote:
> > On 18.09.20 17:48, Soeren Moch wrote:
> >> On 18.09.20 17:21, sbabic at denx.de wrote:
> >>>> This is not required for sysboot (we defined fdtfile), let's save a few
> >>>> bytes in the binary image without these variables.
> >>>> Signed-off-by: Soeren Moch <smoch@web.de>
> >>>> Reviewed-by: Fabio Estevam <festevam@gmail.com>
> >>> Applied to u-boot-imx, master, thanks !
> >>>
> >> Thanks, Stefano.
> >>
> >> What happened to the first patch of this series?? Patchwork status is
> >> "Changes Requested", but his patch only received 2 reviewed-by, no other
> >> requests.
> > Mmmhhh...when patch is in "Changes requested", it disappears from
> > Patchwork's TODO list and I do not see it anymore. Thanks for reporting
> > this, I search for it and I'll pick it up.
> >
> OK, thanks. No problem to apply patch 1/2 after 2/2.
Sorry, I think I see what happened. I had intended to, I suspect, mark
https://patchwork.ozlabs.org/project/uboot/patch/20191010230048.10935-1-smoch at web.de/
as "Changes Requested" (in part because it was suggested to also change
the level of the message, and in part because I think it's been
superseded by a change to do the same thing basically by now) and my
mouse slipped.
I'll re-confirm that the MAC warning thing has been taken care of today,
as well. Again, sorry about that!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200918/a17da9bd/attachment.sig>
next prev parent reply other threads:[~2020-09-18 16:21 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-18 15:48 [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig Soeren Moch
2020-09-18 15:51 ` Stefano Babic
2020-09-18 15:56 ` Soeren Moch
2020-09-18 16:21 ` Tom Rini [this message]
2020-09-18 16:40 ` Soeren Moch
-- strict thread matches above, loose matches on Subject: below --
2020-08-27 19:52 [PATCH 1/2] board: tbs2910: Use recommended distroboot addresses Soeren Moch
2020-08-27 19:52 ` [PATCH 2/2] board: tbs2910: Disable CONFIG_ENV_VARS_UBOOT_CONFIG in defconfig Soeren Moch
2020-08-27 21:24 ` Fabio Estevam
2020-09-18 14:06 ` sbabic at denx.de
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=20200918162156.GV14816@bill-the-cat \
--to=trini@konsulko.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.