From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: "Rafał Miłecki" <zajec5@gmail.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
"Conor Dooley" <conor+dt@kernel.org>
Cc: "Chen-Yu Tsai" <wenst@chromium.org>,
"Hsin-Yi Wang" <hsinyi@chromium.org>,
"Nícolas F . R . A . Prado" <nfraprado@collabora.com>,
"Heiko Stuebner" <heiko.stuebner@cherry.de>,
"Jernej Skrabec" <jernej.skrabec@gmail.com>,
"Chris Morgan" <macromorgan@hotmail.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Sean Wang" <sean.wang@mediatek.com>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
"Rafał Miłecki" <rafal@milecki.pl>
Subject: Re: [PATCH V2 4/4] arm64: dts: mediatek: Add Cudy WR3000 V1
Date: Mon, 26 Feb 2024 09:58:50 +0100 [thread overview]
Message-ID: <9c114a61-ad19-4852-96c8-dbcbe8d8e38c@collabora.com> (raw)
In-Reply-To: <20240223101709.30072-5-zajec5@gmail.com>
Il 23/02/24 11:17, Rafał Miłecki ha scritto:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> Cudy WR3000 V1 is an MT7981B (AKA Filogic 820) based wireless router. It
> has 256 MiB of RAM, some LEDs & buttons and (not described yet) 4
> Ethernet ports.
>
> There is one slightly uncommon LED with a label of globe icon described
> in the Quick Installation Guide as having 2 meanings:
> 1. "Not connected to the Internet"
> 2. "Connected to the Internet"
>
> Vendor designed it to light up when some Internet-accessible IP can be
> reached. This change will be follow by an attempt to define some
> predefined FUNCTION for it.
Can we please try to add that predefined function before adding this devicetree
which would anyway be modified again to use that new definition?
That just makes more sense...
...if you can't get the new definition picked, I will take this one as is, but
please try that first.
Cheers,
Angelo
prev parent reply other threads:[~2024-02-26 8:58 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-23 10:17 [PATCH V2 0/4] dts: mediatek: add Cudy WR3000 V1 wireless router Rafał Miłecki
2024-02-23 10:17 ` [PATCH V2 1/4] dt-bindings: vendor-prefixes: add Cudy Rafał Miłecki
2024-02-26 8:56 ` AngeloGioacchino Del Regno
2024-02-23 10:17 ` [PATCH V2 2/4] dt-bindings: arm64: dts: mediatek: Add Cudy WR3000 V1 router Rafał Miłecki
2024-02-23 10:17 ` [PATCH V2 3/4] arm64: dts: mediatek: mt7981: add pinctrl Rafał Miłecki
2024-02-26 8:56 ` AngeloGioacchino Del Regno
2024-02-23 10:17 ` [PATCH V2 4/4] arm64: dts: mediatek: Add Cudy WR3000 V1 Rafał Miłecki
2024-02-26 8:58 ` AngeloGioacchino Del Regno [this message]
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=9c114a61-ad19-4852-96c8-dbcbe8d8e38c@collabora.com \
--to=angelogioacchino.delregno@collabora.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=heiko.stuebner@cherry.de \
--cc=hsinyi@chromium.org \
--cc=jernej.skrabec@gmail.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=macromorgan@hotmail.com \
--cc=matthias.bgg@gmail.com \
--cc=nfraprado@collabora.com \
--cc=rafal@milecki.pl \
--cc=robh@kernel.org \
--cc=sean.wang@mediatek.com \
--cc=wenst@chromium.org \
--cc=zajec5@gmail.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