Devicetree
 help / color / mirror / Atom feed
From: Oleksij Rempel <o.rempel@pengutronix.de>
To: Jonas Jelonek <jelonek.jonas@gmail.com>
Cc: "Sander Vanheule" <sander@svanheule.net>,
	"Kory Maincent" <kory.maincent@bootlin.com>,
	"Andrew Lunn" <andrew+netdev@lunn.ch>,
	"David S . Miller" <davem@davemloft.net>,
	"Eric Dumazet" <edumazet@google.com>,
	"Jakub Kicinski" <kuba@kernel.org>,
	"Paolo Abeni" <pabeni@redhat.com>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	netdev@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	"Daniel Golle" <daniel@makrotopia.org>,
	"Bjørn Mork" <bjorn@mork.no>,
	"Conor Dooley" <conor.dooley@microchip.com>
Subject: Re: [PATCH net-next v8 1/4] dt-bindings: net: pse-pd: add bindings for Realtek PSE MCU
Date: Fri, 24 Jul 2026 09:44:15 +0200	[thread overview]
Message-ID: <amMXz_KbLyiwB0wz@pengutronix.de> (raw)
In-Reply-To: <e7f50ff9-b5e3-4247-bef3-c0cc3581a50b@gmail.com>

On Fri, Jul 24, 2026 at 09:18:45AM +0200, Jonas Jelonek wrote:
> Hi Oleksij,
> 
> On 23.07.26 12:20, Oleksij Rempel wrote:
> > On Thu, Jul 23, 2026 at 11:45:20AM +0200, Jonas Jelonek wrote:
> >>> Since we already talk about power budgeting - all of this make sense if
> >>> we have port priorities. For example PD692x0 firmware has default prios
> >>> bound to port numbers. Is it the same with this firmware?
> >> Looking at several switches, by default all ports have a priority of 0 assigned.
> >> Thus, no pre-defined prioritisation, it's up to the user to set this.
> > Do this switches have PSU covering all port at max load or it is over
> > provisioning by design. 
> 
> In most cases it's overprovisioned by design, the PSU budget is usually lower
> than what would be needed for all ports at max load. Even one of my switches,
> having a PoE budget of 400W, has 8 ports with 32W and 16 ports with 60 W,
> managed by 5 managers. Usually a manager is not overprovisioned but the
> system PSU is.
> 
> > Haw it behave in reality If we attach more
> > consumer then it is able to handle - all ports go off, or only some of
> > them?
> 
> That's a good question ^^. Unfortunately, I do not have enough PoE-powered
> devices here to test this scenario, or at least it takes a bit of time to get
> enough devices.
> 
> >
> > PD692x0 firmware has user configurable prios - but if many ports share
> > same prios it has internal prios based on port number.
> >
> 
> At least I'm not aware of that but since the firmware on the MCU is a
> blackbox, in the end this could be the case too. But it's not an obvious
> feature.

It would be nice to have both of this answers, but they are not blocker
for this patch set.

> I'd like to slightly adjust the bindings, given the confusion about the naming
> that came up with Sander.

Ack.

> Is there anything else that should be changed? Or is everything else in
> terms of supply, budgeting, etc. fine for follow-ups?

Ack, can be done in follow-ups.

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

  reply	other threads:[~2026-07-24  7:44 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-15  7:55 [PATCH net-next v8 0/4] net: pse-pd: add Realtek PSE MCU support Jonas Jelonek
2026-07-15  7:55 ` [PATCH net-next v8 1/4] dt-bindings: net: pse-pd: add bindings for Realtek PSE MCU Jonas Jelonek
2026-07-17  9:39   ` Kory Maincent
2026-07-22 17:49   ` Sander Vanheule
2026-07-22 19:49     ` Jonas Jelonek
2026-07-23  7:36       ` Oleksij Rempel
2026-07-23  9:45         ` Jonas Jelonek
2026-07-23 10:20           ` Oleksij Rempel
2026-07-24  7:18             ` Jonas Jelonek
2026-07-24  7:44               ` Oleksij Rempel [this message]
2026-07-15  7:55 ` [PATCH net-next v8 2/4] net: pse-pd: add Realtek PSE MCU core Jonas Jelonek
2026-07-17  9:41   ` Kory Maincent
2026-07-15  7:55 ` [PATCH net-next v8 3/4] net: pse-pd: realtek-pse-mcu: add I2C transport Jonas Jelonek
2026-07-17  9:45   ` Kory Maincent
2026-07-15  7:55 ` [PATCH net-next v8 4/4] net: pse-pd: realtek-pse-mcu: add UART transport Jonas Jelonek
2026-07-17  9:46   ` Kory Maincent

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=amMXz_KbLyiwB0wz@pengutronix.de \
    --to=o.rempel@pengutronix.de \
    --cc=andrew+netdev@lunn.ch \
    --cc=bjorn@mork.no \
    --cc=conor+dt@kernel.org \
    --cc=conor.dooley@microchip.com \
    --cc=daniel@makrotopia.org \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=edumazet@google.com \
    --cc=jelonek.jonas@gmail.com \
    --cc=kory.maincent@bootlin.com \
    --cc=krzk+dt@kernel.org \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=robh@kernel.org \
    --cc=sander@svanheule.net \
    /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