From: Rob Herring <robh@kernel.org>
To: Adam Ford <aford173@gmail.com>
Cc: Hugues Fruchet <hugues.fruchet@foss.st.com>,
Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>,
Philipp Zabel <p.zabel@pengutronix.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hverkuil@xs4all.nl>,
linux-media@vger.kernel.org,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
linux-stm32@st-md-mailman.stormreply.com,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org,
Andrzej Pietrasiewicz <andrzej.p@collabora.com>
Subject: Re: [PATCH 3/7] dt-bindings: media: Document STM32MP25 VENC video encoder
Date: Fri, 6 Oct 2023 11:27:03 -0500 [thread overview]
Message-ID: <20231006162703.GA4030032-robh@kernel.org> (raw)
In-Reply-To: <CAHCN7xKrriTPaRMJ-r86cSgFDUUP1At08imLBr_zEP0g3fga_g@mail.gmail.com>
On Wed, Oct 04, 2023 at 06:41:09PM -0500, Adam Ford wrote:
> On Wed, Oct 4, 2023 at 4:16 AM Hugues Fruchet
> <hugues.fruchet@foss.st.com> wrote:
> >
> > Add STM32MP25 VENC video encoder bindings.
> >
> > Signed-off-by: Hugues Fruchet <hugues.fruchet@foss.st.com>
> > ---
> > .../bindings/media/st,stm32mp25-venc.yaml | 56 +++++++++++++++++++
> > 1 file changed, 56 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/media/st,stm32mp25-venc.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/media/st,stm32mp25-venc.yaml b/Documentation/devicetree/bindings/media/st,stm32mp25-venc.yaml
> > new file mode 100644
> > index 000000000000..c69e0a34f675
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/media/st,stm32mp25-venc.yaml
> > @@ -0,0 +1,56 @@
> > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> > +
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/media/st,stm32mp25-venc.yaml#
>
> Can this dt-binding be made more generic, like something like
> hantro-h1 or VC8000NanoE?
>
> I think there will be more boards that may incorporate the Hantro-H1
> or a VC8000 in the future, because I don't think this IP is unique to
> the STM32MP25.
Unless the underlying IP is well documented (i.e. public), then it's
kind of pointless. Everyone will just invent their own numbers and names
of clocks, resets, etc. unless someone can enforce not doing that.
Rob
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-10-06 16:27 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-04 9:15 [PATCH 0/7] Add support for video hardware codec of STMicroelectronics STM32 SoC series Hugues Fruchet
2023-10-04 9:15 ` [PATCH 1/7] dt-bindings: media: Document STM32MP25 VDEC video decoder Hugues Fruchet
2023-10-05 19:45 ` Krzysztof Kozlowski
2023-10-09 13:10 ` Hugues FRUCHET
2023-10-04 9:15 ` [PATCH 2/7] media: hantro: add support for STM32MP25 VDEC Hugues Fruchet
2023-10-11 15:21 ` Nicolas Dufresne
2023-10-04 9:15 ` [PATCH 3/7] dt-bindings: media: Document STM32MP25 VENC video encoder Hugues Fruchet
2023-10-04 23:41 ` Adam Ford
2023-10-05 7:44 ` Hugues FRUCHET
2023-10-06 16:27 ` Rob Herring [this message]
2023-10-09 13:06 ` Hugues FRUCHET
2023-10-05 19:45 ` Krzysztof Kozlowski
2023-10-09 13:49 ` Hugues FRUCHET
2023-10-09 13:56 ` Krzysztof Kozlowski
2023-10-09 14:24 ` Hugues FRUCHET
2023-10-09 14:28 ` Krzysztof Kozlowski
2023-10-09 15:56 ` Hugues FRUCHET
2023-10-04 9:15 ` [PATCH 4/7] media: hantro: add support for STM32MP25 VENC Hugues Fruchet
2023-10-04 9:15 ` [PATCH 5/7] arm64: dts: st: add soc & rifsc structure to stm32mp255 Hugues Fruchet
2023-10-05 19:46 ` Krzysztof Kozlowski
2023-10-09 13:54 ` Hugues FRUCHET
2023-10-04 9:15 ` [PATCH 6/7] arm64: dts: st: add video decoder support " Hugues Fruchet
2023-10-04 9:15 ` [PATCH 7/7] arm64: dts: st: add video encoder " Hugues Fruchet
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=20231006162703.GA4030032-robh@kernel.org \
--to=robh@kernel.org \
--cc=aford173@gmail.com \
--cc=alexandre.torgue@foss.st.com \
--cc=andrzej.p@collabora.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=hugues.fruchet@foss.st.com \
--cc=hverkuil@xs4all.nl \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=mchehab@kernel.org \
--cc=mcoquelin.stm32@gmail.com \
--cc=p.zabel@pengutronix.de \
/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;
as well as URLs for NNTP newsgroup(s).