From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: "Frank Wunderlich" <linux@fw-web.de>,
"MyungJoo Ham" <myungjoo.ham@samsung.com>,
"Kyungmin Park" <kyungmin.park@samsung.com>,
"Chanwoo Choi" <cw00.choi@samsung.com>,
"Georgi Djakov" <djakov@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Andrew Lunn" <andrew@lunn.ch>,
"Vladimir Oltean" <olteanv@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
"Eric Dumazet" <edumazet@google.com>,
"Jakub Kicinski" <kuba@kernel.org>,
"Paolo Abeni" <pabeni@redhat.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"Johnson Wang" <johnson.wang@mediatek.com>,
"Arınç ÜNAL" <arinc.unal@arinc9.com>,
"Landen Chao" <Landen.Chao@mediatek.com>,
"DENG Qingfang" <dqfext@gmail.com>,
"Sean Wang" <sean.wang@mediatek.com>,
"Daniel Golle" <daniel@makrotopia.org>,
"Lorenzo Bianconi" <lorenzo@kernel.org>,
"Felix Fietkau" <nbd@nbd.name>
Cc: Frank Wunderlich <frank-w@public-files.de>,
linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org
Subject: Re: [PATCH v8 04/16] dt-bindings: net: mediatek,net: add sram property
Date: Mon, 7 Jul 2025 12:12:23 +0200 [thread overview]
Message-ID: <f4d046ea-f599-4b4b-a50b-bae00f9f3996@collabora.com> (raw)
In-Reply-To: <20250706132213.20412-5-linux@fw-web.de>
Il 06/07/25 15:21, Frank Wunderlich ha scritto:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> Meditak Filogic SoCs (MT798x) have dedicated MMIO-SRAM for dma operations.
>
> MT7981 and MT7986 currently use static offset to ethernet MAC register
> which will be changed in separate patch once this way is accepted.
>
> Add "sram" property to map ethernet controller to dedicated mmio-sram node.
>
> Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
Honestly, I was more comfortable adding the sram to the reg list instead,
but I know that you got a negative review for that. Oh well.
Please disallow the sram property on SoCs that don't have the SRAM (sram: false),
after which:
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
next prev parent reply other threads:[~2025-07-07 10:12 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-06 13:21 [PATCH v8 00/16] further mt7988 devicetree work Frank Wunderlich
2025-07-06 13:21 ` [PATCH v8 01/16] dt-bindings: net: mediatek,net: update mac subnode pattern for mt7988 Frank Wunderlich
2025-07-07 6:27 ` Krzysztof Kozlowski
2025-07-06 13:21 ` [PATCH v8 02/16] dt-bindings: net: mediatek,net: allow up to 8 IRQs Frank Wunderlich
2025-07-07 6:31 ` Krzysztof Kozlowski
2025-07-07 7:30 ` Frank Wunderlich
2025-07-07 10:06 ` AngeloGioacchino Del Regno
2025-07-07 10:43 ` Frank Wunderlich
2025-07-14 7:41 ` Krzysztof Kozlowski
2025-07-06 13:21 ` [PATCH v8 03/16] dt-bindings: net: mediatek,net: allow irq names Frank Wunderlich
2025-07-07 10:12 ` AngeloGioacchino Del Regno
2025-07-06 13:21 ` [PATCH v8 04/16] dt-bindings: net: mediatek,net: add sram property Frank Wunderlich
2025-07-07 6:33 ` Krzysztof Kozlowski
2025-07-07 10:12 ` AngeloGioacchino Del Regno [this message]
2025-07-06 13:22 ` [PATCH v8 05/16] dt-bindings: net: dsa: mediatek,mt7530: add dsa-port definition for mt7988 Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 06/16] dt-bindings: net: dsa: mediatek,mt7530: add internal mdio bus Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 07/16] dt-bindings: interconnect: add mt7988-cci compatible Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 08/16] arm64: dts: mediatek: mt7988: add cci node Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 09/16] arm64: dts: mediatek: mt7988: add basic ethernet-nodes Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 10/16] arm64: dts: mediatek: mt7988: add switch node Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 11/16] arm64: dts: mediatek: mt7988a-bpi-r4: add proc-supply for cci Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 12/16] arm64: dts: mediatek: mt7988a-bpi-r4: drop unused pins Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 13/16] arm64: dts: mediatek: mt7988a-bpi-r4: add gpio leds Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 14/16] arm64: dts: mediatek: mt7988a-bpi-r4: add aliases for ethernet Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 15/16] arm64: dts: mediatek: mt7988a-bpi-r4: add sfp cages and link to gmac Frank Wunderlich
2025-07-06 13:22 ` [PATCH v8 16/16] arm64: dts: mediatek: mt7988a-bpi-r4: configure switch phys and leds Frank Wunderlich
2025-07-07 9:22 ` (subset) [PATCH v8 00/16] further mt7988 devicetree work AngeloGioacchino Del Regno
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=f4d046ea-f599-4b4b-a50b-bae00f9f3996@collabora.com \
--to=angelogioacchino.delregno@collabora.com \
--cc=Landen.Chao@mediatek.com \
--cc=andrew@lunn.ch \
--cc=arinc.unal@arinc9.com \
--cc=conor+dt@kernel.org \
--cc=cw00.choi@samsung.com \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=djakov@kernel.org \
--cc=dqfext@gmail.com \
--cc=edumazet@google.com \
--cc=frank-w@public-files.de \
--cc=johnson.wang@mediatek.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=kyungmin.park@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux@fw-web.de \
--cc=lorenzo@kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=myungjoo.ham@samsung.com \
--cc=nbd@nbd.name \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--cc=pabeni@redhat.com \
--cc=robh@kernel.org \
--cc=sean.wang@mediatek.com \
/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
all inboxes | Powered by JetHome®