From: Soeren Moch <smoch@web.de>
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 18:40:18 +0200 [thread overview]
Message-ID: <4ee7aa10-ab49-9789-ffec-caeaee24dfcc@web.de> (raw)
In-Reply-To: <20200918162156.GV14816@bill-the-cat>
On 18.09.20 18:21, Tom Rini wrote:
> 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!
>
The MAC warning on tbs2910 was gone some time ago. So the linked patch
must indeed be superseded by something that is already applied.
Soeren
next prev parent reply other threads:[~2020-09-18 16:40 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
2020-09-18 16:40 ` Soeren Moch [this message]
-- 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=4ee7aa10-ab49-9789-ffec-caeaee24dfcc@web.de \
--to=smoch@web.de \
--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