From: "Marek Behún" <marek.behun@nic.cz>
To: Stefan Roese <sr@denx.de>
Cc: u-boot@lists.denx.de, Ramon Fried <rfried.dev@gmail.com>,
Joe Hershberger <joe.hershberger@ni.com>,
Aaron Williams <awilliams@marvell.com>,
Chandrakala Chavva <cchavva@marvell.com>
Subject: Re: [PATCH 1/2] net: phy: marvell: Support reg config via "marvell, reg-init" DT property
Date: Wed, 30 Mar 2022 15:48:04 +0200 [thread overview]
Message-ID: <20220330154804.45cfb42a@thinkpad> (raw)
In-Reply-To: <60e09a5a-60b4-9450-9f60-5be1f32880af@denx.de>
On Wed, 30 Mar 2022 15:33:34 +0200
Stefan Roese <sr@denx.de> wrote:
> So how to continue? Add this "reg-init" "feature" for now? And perhaps
> drop it once this LED configuration is possible via the way you describe
> above? Or drop it once it's also dropped in the Kernel? My personal
> feeling is, that supporting "reg-init" does not "hurt" and might even be
> handy for some testing / debugging of network PHY setups.
Add it now, maybe we will drop it sometime in future if no dts uses it
anymore.
Kernel will need to support it forever because of backwards
compatibility with older DTBs.
Marek
prev parent reply other threads:[~2022-03-30 13:48 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-30 8:38 [PATCH 1/2] net: phy: marvell: Support reg config via "marvell, reg-init" DT property Stefan Roese
2022-03-30 8:38 ` [PATCH 2/2] net: phy: marvell: Add support for 88E1240 PHY Stefan Roese
2022-03-30 12:16 ` [PATCH 1/2] net: phy: marvell: Support reg config via "marvell, reg-init" DT property Marek Behún
2022-03-30 13:33 ` Stefan Roese
2022-03-30 13:48 ` Marek Behún [this message]
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=20220330154804.45cfb42a@thinkpad \
--to=marek.behun@nic.cz \
--cc=awilliams@marvell.com \
--cc=cchavva@marvell.com \
--cc=joe.hershberger@ni.com \
--cc=rfried.dev@gmail.com \
--cc=sr@denx.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