All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felix Fietkau <nbd@nbd.name>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Arnd Bergmann <arnd@arndb.de>
Cc: Networking <netdev@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Lorenzo Bianconi <lorenzo@kernel.org>,
	DTML <devicetree@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	"moderated list:ARM/Mediatek SoC..." 
	<linux-mediatek@lists.infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 04/14] dt-bindings: arm: mediatek: document WED binding for MT7622
Date: Thu, 7 Apr 2022 18:59:46 +0200	[thread overview]
Message-ID: <15c8aab5-41b7-4d25-4b4c-98536bc197fc@nbd.name> (raw)
In-Reply-To: <750c1f9e-6a53-16d5-390e-f9f81fa23afd@linaro.org>


On 06.04.22 10:57, Krzysztof Kozlowski wrote:
> Thanks for clarification. I still wonder about the missing drivers as I
> responded to your second bindings:
> https://lore.kernel.org/all/20220405195755.10817-1-nbd@nbd.name/T/#m6d108c644f0c05cd12c05e56abe2ef75760c6cef
> 
> Both of these compatibles - WED and PCIe - are not actually used. Now
> everything is done inside your Ethernet driver which pokes WED and
> PCIe-mirror address space via regmap/syscon.
> 
> Separate bindings might have sense if WED/PCIe mirror were ever
> converted to real drivers.I think in terms of hardware description it makes more sense to have 
separate nodes, even if the implementation uses them in one driver at 
the moment.

- Felix

WARNING: multiple messages have this Message-ID (diff)
From: Felix Fietkau <nbd@nbd.name>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Arnd Bergmann <arnd@arndb.de>
Cc: Networking <netdev@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Lorenzo Bianconi <lorenzo@kernel.org>,
	DTML <devicetree@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	"moderated list:ARM/Mediatek SoC..."
	<linux-mediatek@lists.infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 04/14] dt-bindings: arm: mediatek: document WED binding for MT7622
Date: Thu, 7 Apr 2022 18:59:46 +0200	[thread overview]
Message-ID: <15c8aab5-41b7-4d25-4b4c-98536bc197fc@nbd.name> (raw)
In-Reply-To: <750c1f9e-6a53-16d5-390e-f9f81fa23afd@linaro.org>


On 06.04.22 10:57, Krzysztof Kozlowski wrote:
> Thanks for clarification. I still wonder about the missing drivers as I
> responded to your second bindings:
> https://lore.kernel.org/all/20220405195755.10817-1-nbd@nbd.name/T/#m6d108c644f0c05cd12c05e56abe2ef75760c6cef
> 
> Both of these compatibles - WED and PCIe - are not actually used. Now
> everything is done inside your Ethernet driver which pokes WED and
> PCIe-mirror address space via regmap/syscon.
> 
> Separate bindings might have sense if WED/PCIe mirror were ever
> converted to real drivers.I think in terms of hardware description it makes more sense to have 
separate nodes, even if the implementation uses them in one driver at 
the moment.

- Felix

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Felix Fietkau <nbd@nbd.name>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Arnd Bergmann <arnd@arndb.de>
Cc: Networking <netdev@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Lorenzo Bianconi <lorenzo@kernel.org>,
	DTML <devicetree@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	"moderated list:ARM/Mediatek SoC..."
	<linux-mediatek@lists.infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 04/14] dt-bindings: arm: mediatek: document WED binding for MT7622
Date: Thu, 7 Apr 2022 18:59:46 +0200	[thread overview]
Message-ID: <15c8aab5-41b7-4d25-4b4c-98536bc197fc@nbd.name> (raw)
In-Reply-To: <750c1f9e-6a53-16d5-390e-f9f81fa23afd@linaro.org>


On 06.04.22 10:57, Krzysztof Kozlowski wrote:
> Thanks for clarification. I still wonder about the missing drivers as I
> responded to your second bindings:
> https://lore.kernel.org/all/20220405195755.10817-1-nbd@nbd.name/T/#m6d108c644f0c05cd12c05e56abe2ef75760c6cef
> 
> Both of these compatibles - WED and PCIe - are not actually used. Now
> everything is done inside your Ethernet driver which pokes WED and
> PCIe-mirror address space via regmap/syscon.
> 
> Separate bindings might have sense if WED/PCIe mirror were ever
> converted to real drivers.I think in terms of hardware description it makes more sense to have 
separate nodes, even if the implementation uses them in one driver at 
the moment.

- Felix

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-04-07 17:01 UTC|newest]

Thread overview: 138+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-05 19:57 [PATCH v2 00/14] MediaTek SoC flow offload improvements + wireless support Felix Fietkau
2022-04-05 19:57 ` Felix Fietkau
2022-04-05 19:57 ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 01/14] dt-bindings: net: mediatek: add optional properties for the SoC ethernet core Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-07 17:20   ` Rob Herring
2022-04-07 17:20     ` Rob Herring
2022-04-07 17:20     ` Rob Herring
2022-04-08  9:34     ` Lorenzo Bianconi
2022-04-08  9:34       ` Lorenzo Bianconi
2022-04-08  9:34       ` Lorenzo Bianconi
2022-04-05 19:57 ` [PATCH v2 02/14] net: ethernet: mtk_eth_soc: add support for coherent DMA Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 03/14] arm64: dts: mediatek: mt7622: " Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 04/14] dt-bindings: arm: mediatek: document WED binding for MT7622 Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-06  8:09   ` Krzysztof Kozlowski
2022-04-06  8:09     ` Krzysztof Kozlowski
2022-04-06  8:09     ` Krzysztof Kozlowski
2022-04-06  8:18     ` Felix Fietkau
2022-04-06  8:18       ` Felix Fietkau
2022-04-06  8:18       ` Felix Fietkau
2022-04-06  8:29       ` Arnd Bergmann
2022-04-06  8:29         ` Arnd Bergmann
2022-04-06  8:29         ` Arnd Bergmann
2022-04-06  8:32         ` Felix Fietkau
2022-04-06  8:32           ` Felix Fietkau
2022-04-06  8:32           ` Felix Fietkau
2022-04-06  8:57           ` Krzysztof Kozlowski
2022-04-06  8:57             ` Krzysztof Kozlowski
2022-04-06  8:57             ` Krzysztof Kozlowski
2022-04-07 16:59             ` Felix Fietkau [this message]
2022-04-07 16:59               ` Felix Fietkau
2022-04-07 16:59               ` Felix Fietkau
2022-04-07 15:50       ` Andrew Lunn
2022-04-07 15:50         ` Andrew Lunn
2022-04-07 15:50         ` Andrew Lunn
2022-04-07 16:10         ` Felix Fietkau
2022-04-07 16:10           ` Felix Fietkau
2022-04-07 16:10           ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 05/14] dt-bindings: arm: mediatek: document the pcie mirror node on MT7622 Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-06  8:20   ` Krzysztof Kozlowski
2022-04-06  8:20     ` Krzysztof Kozlowski
2022-04-06  8:20     ` Krzysztof Kozlowski
2022-04-06 11:01     ` Felix Fietkau
2022-04-06 11:01       ` Felix Fietkau
2022-04-06 11:01       ` Felix Fietkau
2022-04-07 17:16       ` Rob Herring
2022-04-07 17:16         ` Rob Herring
2022-04-07 17:16         ` Rob Herring
2022-04-07 17:29         ` Felix Fietkau
2022-04-07 17:29           ` Felix Fietkau
2022-04-07 17:29           ` Felix Fietkau
2022-04-07 17:19   ` Rob Herring
2022-04-07 17:19     ` Rob Herring
2022-04-07 17:19     ` Rob Herring
2022-04-08  9:03     ` Felix Fietkau
2022-04-08  9:03       ` Felix Fietkau
2022-04-08  9:03       ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 06/14] net: ethernet: mtk_eth_soc: add support for Wireless Ethernet Dispatch (WED) Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 07/14] net: ethernet: mtk_eth_soc: implement flow offloading to WED devices Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 08/14] arm64: dts: mediatek: mt7622: introduce nodes for Wireless Ethernet Dispatch Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 09/14] net: ethernet: mtk_eth_soc: add ipv6 flow offload support Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 10/14] net: ethernet: mtk_eth_soc: support TC_SETUP_BLOCK for PPE offload Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 11/14] net: ethernet: mtk_eth_soc: allocate struct mtk_ppe separately Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 12/14] net: ethernet: mtk_eth_soc: rework hardware flow table management Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 13/14] net: ethernet: mtk_eth_soc: remove bridge flow offload type entry support Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57 ` [PATCH v2 14/14] net: ethernet: mtk_eth_soc: support creating mac address based offload entries Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-05 19:57   ` Felix Fietkau
2022-04-07 18:10   ` Andrew Lunn
2022-04-07 18:10     ` Andrew Lunn
2022-04-07 18:10     ` Andrew Lunn
2022-04-07 18:21     ` Felix Fietkau
2022-04-07 18:21       ` Felix Fietkau
2022-04-07 18:21       ` Felix Fietkau
2022-04-11 13:00       ` Andrew Lunn
2022-04-11 13:00         ` Andrew Lunn
2022-04-11 13:00         ` Andrew Lunn
2022-04-12  7:13         ` Felix Fietkau
2022-04-12  7:13           ` Felix Fietkau
2022-04-12  7:13           ` Felix Fietkau
2022-04-12 13:07           ` Andrew Lunn
2022-04-12 13:07             ` Andrew Lunn
2022-04-12 13:07             ` Andrew Lunn
2022-04-12 13:49             ` Felix Fietkau
2022-04-12 13:49               ` Felix Fietkau
2022-04-12 13:49               ` Felix Fietkau
2022-04-12 14:21               ` Andrew Lunn
2022-04-12 14:21                 ` Andrew Lunn
2022-04-12 14:21                 ` Andrew Lunn
2022-04-12 15:51                 ` Felix Fietkau
2022-04-12 15:51                   ` Felix Fietkau
2022-04-12 15:51                   ` Felix Fietkau
2022-04-12 17:37                   ` Andrew Lunn
2022-04-12 17:37                     ` Andrew Lunn
2022-04-12 17:37                     ` Andrew Lunn
2022-04-12 17:51                     ` Felix Fietkau
2022-04-12 17:51                       ` Felix Fietkau
2022-04-12 17:51                       ` Felix Fietkau
2022-04-06 13:30 ` [PATCH v2 00/14] MediaTek SoC flow offload improvements + wireless support patchwork-bot+netdevbpf
2022-04-06 13:30   ` patchwork-bot+netdevbpf
2022-04-06 13:30   ` patchwork-bot+netdevbpf
2022-04-07 15:57   ` Andrew Lunn
2022-04-07 15:57     ` Andrew Lunn
2022-04-07 15:57     ` Andrew Lunn
2022-04-07 17:00     ` Felix Fietkau
2022-04-07 17:00       ` Felix Fietkau
2022-04-07 17:00       ` Felix Fietkau
2022-04-07 17:28       ` Andrew Lunn
2022-04-07 17:28         ` Andrew Lunn
2022-04-07 17:28         ` Andrew Lunn
2022-04-07 17:34         ` Felix Fietkau
2022-04-07 17:34           ` Felix Fietkau
2022-04-07 17:34           ` Felix Fietkau

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=15c8aab5-41b7-4d25-4b4c-98536bc197fc@nbd.name \
    --to=nbd@nbd.name \
    --cc=arnd@arndb.de \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=kuba@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=lorenzo@kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=robh+dt@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.