mbox series

[net-next,0/5] dt-bindings: net: realtek,rtl9301-switch

Message ID 20250209234751.460404-1-chris.packham@alliedtelesis.co.nz (mailing list archive)
Headers show
Series dt-bindings: net: realtek,rtl9301-switch | expand

Message

Chris Packham Feb. 9, 2025, 11:47 p.m. UTC
This is my attempt at trying to sort out the mess I've created with the RTL9300
switch dt-bindings. Some context is available on [1] and [2].

The first patch just moves the binding from mfd/ to net/ (with an adjustment of
the internal path name). The next two patches are successors to patches already
sent as part of the series [3]. Finally the last two patches are needed to
update the in-tree dts to pass `make dtbs_check`, I couldn't figure out a clean
way of decoupling these `make dtbs_check` will either complain that the dts has
unexpected properties or that it is missing required properties.

[1] - https://lore.kernel.org/lkml/20250204-eccentric-deer-of-felicity-02b7ee@krzk-bin/
[2] - https://lore.kernel.org/lkml/4e3c5d83-d215-4eff-bf02-6d420592df8f@alliedtelesis.co.nz/
[3] - https://lore.kernel.org/lkml/20250204030249.1965444-1-chris.packham@alliedtelesis.co.nz/

Chris Packham (5):
  dt-bindings: net: Move realtek,rtl9301-switch to net
  dt-bindings: net: Add switch ports and interrupts to RTL9300
  dt-bindings: net: Add Realtek MDIO controller
  mips: dts: realtek: Add switch interrupts
  mips: dts: cameo-rtl9302c: Add switch block

 .../bindings/net/realtek,rtl9301-mdio.yaml    | 86 +++++++++++++++++++
 .../{mfd => net}/realtek,rtl9301-switch.yaml  | 63 +++++++++++++-
 .../cameo-rtl9302c-2x-rtl8224-2xge.dts        | 48 +++++++++++
 arch/mips/boot/dts/realtek/rtl930x.dtsi       |  3 +
 4 files changed, 199 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
 rename Documentation/devicetree/bindings/{mfd => net}/realtek,rtl9301-switch.yaml (66%)

Comments

Jakub Kicinski Feb. 14, 2025, 9:45 p.m. UTC | #1
On Mon, 10 Feb 2025 12:47:46 +1300 Chris Packham wrote:
>   dt-bindings: net: Move realtek,rtl9301-switch to net
>   dt-bindings: net: Add switch ports and interrupts to RTL9300
>   dt-bindings: net: Add Realtek MDIO controller

AFAIU we're waiting for a review from DT maintainers on this one,
is this series on your radar?
Jakub Kicinski Feb. 18, 2025, 2:55 p.m. UTC | #2
On Fri, 14 Feb 2025 13:45:28 -0800 Jakub Kicinski wrote:
> On Mon, 10 Feb 2025 12:47:46 +1300 Chris Packham wrote:
> >   dt-bindings: net: Move realtek,rtl9301-switch to net
> >   dt-bindings: net: Add switch ports and interrupts to RTL9300
> >   dt-bindings: net: Add Realtek MDIO controller  
> 
> AFAIU we're waiting for a review from DT maintainers on this one,
> is this series on your radar?

Not sure why we're getting no reviews here.
Chris, could you repost? Maybe we'll get this back into people's review
queues that way..
Chris Packham Feb. 18, 2025, 7:43 p.m. UTC | #3
Hi Jakub,

On 19/02/2025 03:55, Jakub Kicinski wrote:
> On Fri, 14 Feb 2025 13:45:28 -0800 Jakub Kicinski wrote:
>> On Mon, 10 Feb 2025 12:47:46 +1300 Chris Packham wrote:
>>>    dt-bindings: net: Move realtek,rtl9301-switch to net
>>>    dt-bindings: net: Add switch ports and interrupts to RTL9300
>>>    dt-bindings: net: Add Realtek MDIO controller
>> AFAIU we're waiting for a review from DT maintainers on this one,
>> is this series on your radar?
> Not sure why we're getting no reviews here.
> Chris, could you repost? Maybe we'll get this back into people's review
> queues that way..

I know there have been some issues with gmail rejecting stuff coming 
from our domain. I wasn't sure if it was that or if I'd just exhausted 
the DT maintainers. Possibly both.

I'll send out the series again with a RESEND tag.