From: Tom Rini <trini@konsulko.com>
To: Ramon Fried <rfried.dev@gmail.com>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>
Subject: Re: Please pull u-boot-net/next
Date: Wed, 17 Nov 2021 09:09:27 -0500 [thread overview]
Message-ID: <20211117140927.GY24579@bill-the-cat> (raw)
In-Reply-To: <CAGi-RUJ1+LO0fA41FfREDPAvqGLn0bWB9icE5_HY0-DZaCXLBw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1668 bytes --]
On Tue, Nov 16, 2021 at 08:08:36AM +0200, Ramon Fried wrote:
> On Mon, Nov 15, 2021 at 7:37 PM Tom Rini <trini@konsulko.com> wrote:
> >
> > On Wed, Nov 10, 2021 at 07:58:42PM +0200, Ramon Fried wrote:
> >
> > > his PR includes the following net changes for next:
> > >
> > > - Various DSA additions
> > > - bootp: fix for VCI string
> > > - tsec: support for promiscuous mode
> > > - macb: Fix for random MAC address not saved to the environment.
> > > The following changes since commit e8e9c6f48400989c5fc54467576f8d535bd713e6:
> > >
> > > Merge https://source.denx.de/u-boot/custodians/u-boot-samsung
> > > (2021-11-09 15:05:33 -0500)
> > >
> > > are available in the Git repository at:
> > >
> > > https://source.denx.de/u-boot/custodians/u-boot-net.git/
> > >
> > > for you to fetch changes up to 23a438b2d05db54c95315094fe9ac8fe0c46213e:
> > >
> > > net: uclass: Save ethernet MAC address when generated (2021-11-10
> > > 19:55:01 +0200)
> >
> > As it seems we're still actively discussing the implications of:
> >
> > > Michal Simek (1):
> > > net: uclass: Save ethernet MAC address when generated
> >
> > Can you please re-spin with that dropped? Thanks!
>
> I sure can, but I'm against that. the patch should stay IMO.
> I saw the thread, it actually started in IRC, where I already gave my opinion.
I'm cycling back to this again because setting aside that I'm not
convinced yet this is the right path, it's also an incomplete patch
because the help text for NET_RANDOM_ETHADDR specifically says the
environment is not updated, which is no longer true after that patch and
needs to be corrected.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2021-11-17 14:09 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-10 17:58 Please pull u-boot-net/next Ramon Fried
2021-11-15 17:37 ` Tom Rini
2021-11-16 6:08 ` Ramon Fried
2021-11-17 14:09 ` Tom Rini [this message]
2021-11-23 3:24 ` Ramon Fried
-- strict thread matches above, loose matches on Subject: below --
2022-04-13 15:10 Ramon Fried
2022-04-13 22:57 ` Tom Rini
2022-04-15 15:47 ` Tom Rini
2022-01-15 20:01 Ramon Fried
2022-01-17 16:11 ` Tom Rini
2021-12-02 8:25 Ramon Fried
2021-12-02 14:57 ` Tom Rini
2021-11-23 3:25 Ramon Fried
2021-11-23 3:27 ` Tom Rini
2021-11-23 8:12 ` Ramon Fried
2021-11-03 9:13 Ramon Fried
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=20211117140927.GY24579@bill-the-cat \
--to=trini@konsulko.com \
--cc=rfried.dev@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox