From: Rob Herring <robh@kernel.org>
To: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Jerome Brunet <jbrunet@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Jagan Teki <jagan@amarulasolutions.com>,
Nicolas Belin <nbelin@baylibre.com>,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: drop unneeded reg property and example
Date: Fri, 10 Nov 2023 14:57:16 -0600 [thread overview]
Message-ID: <20231110205716.GA413638-robh@kernel.org> (raw)
In-Reply-To: <20231109-amlogic-v6-4-upstream-dsi-ccf-vim3-v8-4-81e4aeeda193@linaro.org>
On Thu, Nov 09, 2023 at 10:00:05AM +0100, Neil Armstrong wrote:
> Now this bindings is referred from amlogic,meson-gx-hhi-sysctrl.yaml and is
> documented as a subnode of a simple-mfd, drop the invalid reg property.
Why is it invalid? It's preferred to have 'reg' in MFDs even if Linux
doesn't use them. If there's a chunk of registers you can define, then
do so. If it's all register bit soup, then fine, omit it.
Rob
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Jerome Brunet <jbrunet@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Jagan Teki <jagan@amarulasolutions.com>,
Nicolas Belin <nbelin@baylibre.com>,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: drop unneeded reg property and example
Date: Fri, 10 Nov 2023 14:57:16 -0600 [thread overview]
Message-ID: <20231110205716.GA413638-robh@kernel.org> (raw)
In-Reply-To: <20231109-amlogic-v6-4-upstream-dsi-ccf-vim3-v8-4-81e4aeeda193@linaro.org>
On Thu, Nov 09, 2023 at 10:00:05AM +0100, Neil Armstrong wrote:
> Now this bindings is referred from amlogic,meson-gx-hhi-sysctrl.yaml and is
> documented as a subnode of a simple-mfd, drop the invalid reg property.
Why is it invalid? It's preferred to have 'reg' in MFDs even if Linux
doesn't use them. If there's a chunk of registers you can define, then
do so. If it's all register bit soup, then fine, omit it.
Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Jerome Brunet <jbrunet@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Jagan Teki <jagan@amarulasolutions.com>,
Nicolas Belin <nbelin@baylibre.com>,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: drop unneeded reg property and example
Date: Fri, 10 Nov 2023 14:57:16 -0600 [thread overview]
Message-ID: <20231110205716.GA413638-robh@kernel.org> (raw)
In-Reply-To: <20231109-amlogic-v6-4-upstream-dsi-ccf-vim3-v8-4-81e4aeeda193@linaro.org>
On Thu, Nov 09, 2023 at 10:00:05AM +0100, Neil Armstrong wrote:
> Now this bindings is referred from amlogic,meson-gx-hhi-sysctrl.yaml and is
> documented as a subnode of a simple-mfd, drop the invalid reg property.
Why is it invalid? It's preferred to have 'reg' in MFDs even if Linux
doesn't use them. If there's a chunk of registers you can define, then
do so. If it's all register bit soup, then fine, omit it.
Rob
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Neil Armstrong <neil.armstrong@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
Conor Dooley <conor+dt@kernel.org>,
Stephen Boyd <sboyd@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
Jagan Teki <jagan@amarulasolutions.com>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org,
Nicolas Belin <nbelin@baylibre.com>,
Jerome Brunet <jbrunet@baylibre.com>
Subject: Re: [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: drop unneeded reg property and example
Date: Fri, 10 Nov 2023 14:57:16 -0600 [thread overview]
Message-ID: <20231110205716.GA413638-robh@kernel.org> (raw)
In-Reply-To: <20231109-amlogic-v6-4-upstream-dsi-ccf-vim3-v8-4-81e4aeeda193@linaro.org>
On Thu, Nov 09, 2023 at 10:00:05AM +0100, Neil Armstrong wrote:
> Now this bindings is referred from amlogic,meson-gx-hhi-sysctrl.yaml and is
> documented as a subnode of a simple-mfd, drop the invalid reg property.
Why is it invalid? It's preferred to have 'reg' in MFDs even if Linux
doesn't use them. If there's a chunk of registers you can define, then
do so. If it's all register bit soup, then fine, omit it.
Rob
next prev parent reply other threads:[~2023-11-10 20:57 UTC|newest]
Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-09 9:00 [PATCH v8 00/12] drm/meson: add support for MIPI DSI Display Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 01/12] dt-bindings: clk: g12a-clkc: add CTS_ENCL clock ids Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: add example covering meson-axg-hhi-sysctrl Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 17:34 ` Conor Dooley
2023-11-09 17:34 ` [PATCH v8 02/12] dt-bindings: soc: amlogic, meson-gx-hhi-sysctrl: " Conor Dooley
2023-11-09 17:34 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: " Conor Dooley
2023-11-09 17:34 ` Conor Dooley
2023-11-10 7:50 ` Neil Armstrong
2023-11-10 7:50 ` [PATCH v8 02/12] dt-bindings: soc: amlogic, meson-gx-hhi-sysctrl: " Neil Armstrong
2023-11-10 7:50 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: " Neil Armstrong
2023-11-10 7:50 ` Neil Armstrong
2023-11-10 18:26 ` Conor Dooley
2023-11-10 18:26 ` [PATCH v8 02/12] dt-bindings: soc: amlogic, meson-gx-hhi-sysctrl: " Conor Dooley
2023-11-10 18:26 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: " Conor Dooley
2023-11-10 18:26 ` Conor Dooley
2023-11-10 20:12 ` Rob Herring
2023-11-10 20:12 ` [PATCH v8 02/12] dt-bindings: soc: amlogic, meson-gx-hhi-sysctrl: " Rob Herring
2023-11-10 20:12 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: " Rob Herring
2023-11-10 20:12 ` Rob Herring
2023-11-14 14:06 ` Neil Armstrong
2023-11-14 14:06 ` [PATCH v8 02/12] dt-bindings: soc: amlogic, meson-gx-hhi-sysctrl: " Neil Armstrong
2023-11-14 14:06 ` [PATCH v8 02/12] dt-bindings: soc: amlogic,meson-gx-hhi-sysctrl: " Neil Armstrong
2023-11-14 14:06 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 03/12] dt-bindings: phy: amlogic,meson-axg-mipi-pcie-analog: drop text about parent syscon and drop example Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 17:50 ` Conor Dooley
2023-11-09 17:50 ` Conor Dooley
2023-11-09 17:50 ` Conor Dooley
2023-11-09 17:50 ` Conor Dooley
2023-11-09 9:00 ` [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: drop unneeded reg property and example Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 18:04 ` Conor Dooley
2023-11-09 18:04 ` Conor Dooley
2023-11-09 18:04 ` Conor Dooley
2023-11-09 18:04 ` Conor Dooley
2023-11-10 7:51 ` Neil Armstrong
2023-11-10 7:51 ` [PATCH v8 04/12] dt-bindings: phy: amlogic, g12a-mipi-dphy-analog: " Neil Armstrong
2023-11-10 7:51 ` [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: " Neil Armstrong
2023-11-10 7:51 ` Neil Armstrong
2023-11-10 20:57 ` Rob Herring [this message]
2023-11-10 20:57 ` Rob Herring
2023-11-10 20:57 ` Rob Herring
2023-11-10 20:57 ` Rob Herring
2023-11-14 14:08 ` Neil Armstrong
2023-11-14 14:08 ` [PATCH v8 04/12] dt-bindings: phy: amlogic, g12a-mipi-dphy-analog: " Neil Armstrong
2023-11-14 14:08 ` [PATCH v8 04/12] dt-bindings: phy: amlogic,g12a-mipi-dphy-analog: " Neil Armstrong
2023-11-14 14:08 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 05/12] dt-bindings: arm: amlogic: Document the MNT Reform 2 CM4 adapter with a BPI-CM4 Module Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 06/12] clk: meson: g12a: add CTS_ENCL & CTS_ENCL_SEL clocks Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 07/12] clk: meson: add vclk driver Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 08/12] clk: meson: g12a: make VCLK2 and ENCL clock path configurable by CCF Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 09/12] drm/meson: gate px_clk when setting rate Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 10/12] arm64: meson: g12-common: add the MIPI DSI nodes Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 11/12] DONOTMERGE: arm64: meson: khadas-vim3l: add DSI panel Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` [PATCH v8 12/12] arm64: dts: amlogic: meson-g12b-bananapi-cm4: add support for MNT Reform2 with CM4 adaper Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
2023-11-09 9:00 ` Neil Armstrong
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=20231110205716.GA413638-robh@kernel.org \
--to=robh@kernel.org \
--cc=airlied@gmail.com \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jagan@amarulasolutions.com \
--cc=jbrunet@baylibre.com \
--cc=khilman@baylibre.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=mturquette@baylibre.com \
--cc=nbelin@baylibre.com \
--cc=neil.armstrong@linaro.org \
--cc=sboyd@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.