All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kyle Swenson <kyle.swenson@est.tech>
To: Kory Maincent <kory.maincent@bootlin.com>
Cc: "o.rempel@pengutronix.de" <o.rempel@pengutronix.de>,
	"andrew+netdev@lunn.ch" <andrew+netdev@lunn.ch>,
	"davem@davemloft.net" <davem@davemloft.net>,
	"edumazet@google.com" <edumazet@google.com>,
	"kuba@kernel.org" <kuba@kernel.org>,
	"pabeni@redhat.com" <pabeni@redhat.com>,
	"robh@kernel.org" <robh@kernel.org>,
	"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
	"conor+dt@kernel.org" <conor+dt@kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [RFC PATCH net-next 1/2] dt-bindings: net: pse-pd: Describe the LTC4266 PSE chipset
Date: Wed, 4 Jun 2025 14:53:36 +0000	[thread overview]
Message-ID: <aEBd4SMbfFOx4YHE@p620> (raw)
In-Reply-To: <20250604110654.52dd736b@kmaincent-XPS-13-7390>

On Wed, Jun 04, 2025 at 11:06:54AM +0200, Kory Maincent wrote:
> Le Tue, 3 Jun 2025 23:04:37 +0000,
> Kyle Swenson <kyle.swenson@est.tech> a écrit :
> 
> Hello Kyle,
> 
> Nice to see your patch!

Thanks!  A bit later than I had intended but late is better than never I
guess.

> 
> Your Sign-off-by tag is missing on all your patches.

My mistake.  I had intended to add that but evidently didn't.

> > +  channels:
> > +
> > +    description: This parameter describes the mapping between the logical
> > ports
> > +      on the PSE controller and the physical ports.
> 
> The channels parameter is not describing the mapping but only the channels
> list. I also have to change the tps23881 binding doc similarly.
> We discussed about this here
> https://lore.kernel.org/netdev/20250517003525.2f6a5005@kmaincent-XPS-13-7390/

Oh, thanks for the link.  I'll fix this (and all the other issues in the
binding) in v2.

Thanks,
Kyle

  reply	other threads:[~2025-06-04 14:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-03 23:04 [RFC PATCH net-next 0/2] Add support for the LTC4266 PSE Controller Kyle Swenson
2025-06-03 23:04 ` [RFC PATCH net-next 1/2] dt-bindings: net: pse-pd: Describe the LTC4266 PSE chipset Kyle Swenson
2025-06-04  6:43   ` Krzysztof Kozlowski
2025-06-04  9:06   ` Kory Maincent
2025-06-04 14:53     ` Kyle Swenson [this message]
2025-06-03 23:04 ` [RFC PATCH net-next 2/2] net: pse-pd: Add LTC4266 PSE controller driver Kyle Swenson
2025-06-04  8:39   ` Oleksij Rempel
2025-06-09 14:23     ` Kyle Swenson

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=aEBd4SMbfFOx4YHE@p620 \
    --to=kyle.swenson@est.tech \
    --cc=andrew+netdev@lunn.ch \
    --cc=conor+dt@kernel.org \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=edumazet@google.com \
    --cc=kory.maincent@bootlin.com \
    --cc=krzk+dt@kernel.org \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=o.rempel@pengutronix.de \
    --cc=pabeni@redhat.com \
    --cc=robh@kernel.org \
    /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.