All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Gaignard <benjamin.gaignard@collabora.com>
To: Conor Dooley <conor@kernel.org>
Cc: nicolas.dufresne@collabora.com, p.zabel@pengutronix.de,
	mchehab@kernel.org, robh@kernel.org, krzk+dt@kernel.org,
	conor+dt@kernel.org, Frank.Li@nxp.com, s.hauer@pengutronix.de,
	kernel@pengutronix.de, festevam@gmail.com,
	linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@collabora.com,
	Conor Dooley <conor.dooley@microchip.com>
Subject: Re: [PATCH v5 2/3] media: dt-bindings: media: nxp, imx8mq-vpu: Remove nxp,imx8mq-vpu
Date: Tue, 29 Sep 2026 09:32:15 +0200	[thread overview]
Message-ID: <4ed94c10-edc7-423b-a4db-032af5180f3f@collabora.com> (raw)
In-Reply-To: <20260928-primal-figurine-2a87db42eab0@spud>


Le 28/09/2026 à 18:31, Conor Dooley a écrit :
> On Mon, Sep 28, 2026 at 10:08:21AM +0200, Benjamin Gaignard wrote:
>> nxp,imx8mq-vpu has been tagged has deprecated since a while and
>> driver emitted a warning when this bindings was used.
>> Vendor DT use the following node before they moved to mainline driver:
>> vpu: vpu@38300000 {
>> 	compatible = "nxp,imx8mq-hantro";
>> 	...
>> };
>> but that binding has never be documented.
>> After driver clean up it is time to remove it.
>>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
>> Reviewed-by: Frank Li <Frank.Li@nxp.com>
>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>> pw-bot: not-applicable
> This is not a tag that you should put in your commits btw, this was me
> telling the dt patchwork that this patch has been handled.

It was the first time I saw this tad and I didn't know what to do of it
but I found it in some other commit messages so I added it.

>
>> ---
>> changes in v5:
>> - add more details in commit message
>> - add Conor Dooley accked-by tag
>>
>>   .../devicetree/bindings/media/nxp,imx8mq-vpu.yaml      | 10 ++++------
>>   1 file changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml b/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> index 19528262810a..69619d2c86fa 100644
>> --- a/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> +++ b/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> @@ -14,12 +14,10 @@ description:
>>   
>>   properties:
>>     compatible:
>> -    oneOf:
>> -      - const: nxp,imx8mq-vpu
>> -        deprecated: true
>> -      - const: nxp,imx8mq-vpu-g1
>> -      - const: nxp,imx8mq-vpu-g2
>> -      - const: nxp,imx8mm-vpu-g1
>> +    enum:
>> +      - nxp,imx8mq-vpu-g1
>> +      - nxp,imx8mq-vpu-g2
>> +      - nxp,imx8mm-vpu-g1
>>   
>>     reg:
>>       maxItems: 1
>> -- 
>> 2.53.0
>>

WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Gaignard <benjamin.gaignard@collabora.com>
To: Conor Dooley <conor@kernel.org>
Cc: nicolas.dufresne@collabora.com, p.zabel@pengutronix.de,
	mchehab@kernel.org, robh@kernel.org, krzk+dt@kernel.org,
	conor+dt@kernel.org, Frank.Li@nxp.com, s.hauer@pengutronix.de,
	kernel@pengutronix.de, festevam@gmail.com,
	linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, kernel@collabora.com,
	Conor Dooley <conor.dooley@microchip.com>
Subject: Re: [PATCH v5 2/3] media: dt-bindings: media: nxp, imx8mq-vpu: Remove nxp,imx8mq-vpu
Date: Tue, 29 Sep 2026 09:32:15 +0200	[thread overview]
Message-ID: <4ed94c10-edc7-423b-a4db-032af5180f3f@collabora.com> (raw)
In-Reply-To: <20260928-primal-figurine-2a87db42eab0@spud>


Le 28/09/2026 à 18:31, Conor Dooley a écrit :
> On Mon, Sep 28, 2026 at 10:08:21AM +0200, Benjamin Gaignard wrote:
>> nxp,imx8mq-vpu has been tagged has deprecated since a while and
>> driver emitted a warning when this bindings was used.
>> Vendor DT use the following node before they moved to mainline driver:
>> vpu: vpu@38300000 {
>> 	compatible = "nxp,imx8mq-hantro";
>> 	...
>> };
>> but that binding has never be documented.
>> After driver clean up it is time to remove it.
>>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@collabora.com>
>> Reviewed-by: Frank Li <Frank.Li@nxp.com>
>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>> pw-bot: not-applicable
> This is not a tag that you should put in your commits btw, this was me
> telling the dt patchwork that this patch has been handled.

It was the first time I saw this tad and I didn't know what to do of it
but I found it in some other commit messages so I added it.

>
>> ---
>> changes in v5:
>> - add more details in commit message
>> - add Conor Dooley accked-by tag
>>
>>   .../devicetree/bindings/media/nxp,imx8mq-vpu.yaml      | 10 ++++------
>>   1 file changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml b/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> index 19528262810a..69619d2c86fa 100644
>> --- a/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> +++ b/Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
>> @@ -14,12 +14,10 @@ description:
>>   
>>   properties:
>>     compatible:
>> -    oneOf:
>> -      - const: nxp,imx8mq-vpu
>> -        deprecated: true
>> -      - const: nxp,imx8mq-vpu-g1
>> -      - const: nxp,imx8mq-vpu-g2
>> -      - const: nxp,imx8mm-vpu-g1
>> +    enum:
>> +      - nxp,imx8mq-vpu-g1
>> +      - nxp,imx8mq-vpu-g2
>> +      - nxp,imx8mm-vpu-g1
>>   
>>     reg:
>>       maxItems: 1
>> -- 
>> 2.53.0
>>

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

  reply	other threads:[~2026-09-29  7:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-28  8:08 [PATCH v5 0/3] media: verisilicon: remove unused variant and code clean up Benjamin Gaignard
2026-09-28  8:08 ` Benjamin Gaignard
2026-09-28  8:08 ` [PATCH v5 1/3] media: verisilicon: Remove deprecated nxp,imx8mp-vpu variant Benjamin Gaignard
2026-09-28  8:08   ` Benjamin Gaignard
2026-09-28  8:19   ` sashiko-bot
2026-09-28 16:41   ` Frank Li
2026-09-28 16:41     ` Frank Li
2026-09-28 17:38   ` Nicolas Dufresne
2026-09-28 17:38     ` Nicolas Dufresne
2026-09-28  8:08 ` [PATCH v5 2/3] media: dt-bindings: media: nxp, imx8mq-vpu: Remove nxp,imx8mq-vpu Benjamin Gaignard
2026-09-28  8:08   ` Benjamin Gaignard
2026-09-28 16:31   ` Conor Dooley
2026-09-28 16:31     ` Conor Dooley
2026-09-29  7:32     ` Benjamin Gaignard [this message]
2026-09-29  7:32       ` Benjamin Gaignard
2026-09-28  8:08 ` [PATCH v5 3/3] media: verisilicon: Remove useless internal fields Benjamin Gaignard
2026-09-28  8:08   ` Benjamin Gaignard

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=4ed94c10-edc7-423b-a4db-032af5180f3f@collabora.com \
    --to=benjamin.gaignard@collabora.com \
    --cc=Frank.Li@nxp.com \
    --cc=conor+dt@kernel.org \
    --cc=conor.dooley@microchip.com \
    --cc=conor@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=imx@lists.linux.dev \
    --cc=kernel@collabora.com \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.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=mchehab@kernel.org \
    --cc=nicolas.dufresne@collabora.com \
    --cc=p.zabel@pengutronix.de \
    --cc=robh@kernel.org \
    --cc=s.hauer@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 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.