mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Quchaosheng <quchaosheng000406@163.com>
Cc: Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>,
	dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
	linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] dt-bindings: display: ti,tpd12s015: convert, and fix what the conversion exposed
Date: Fri, 9 Oct 2026 13:03:06 +0200	[thread overview]
Message-ID: <20261009110306.GB680229@killaraus.ideasonboard.com> (raw)
In-Reply-To: <20261009105601.301891-1-quchaosheng000406@163.com>

This is the fourth version of this series that you send within 10
minutes, without any version number in the subject line. Please stop
flooding reviewers.

On Fri, Oct 09, 2026 at 06:55:58PM +0800, Quchaosheng wrote:
> From: Chaosheng Qu <quchaosheng000406@163.com>
> 
> v4.  The three things Laurent pointed out on the v3 thread are addressed,
> and the conversion is separated from the two devicetree fixes it turned
> up.
> 
>   [1/3] converts ti,tpd12s015.txt to DT schema.  gpios is back to three
>         entries: the two drivers obtain exactly three through
>         devm_gpiod_get_index(), indices 0 to 2, and neither file mentions
>         scl, sda or ddc.  v3 had raised maxItems to five only to keep
>         dtbs_check quiet for omap5-igep0050.dts, which wrote a defect of
>         the DT source into the binding.  That was backwards.
> 
>   [2/3] drops the two extra entries from omap5-igep0050.dts, which is
>         where the defect belongs.  Nothing reads them: there is no
>         pinctrl group for those lines and no other consumer in the tree.
> 
>   [3/3] drops the unit-address from the tpd12s015 endpoints on
>         am5729-beagleboneai and am57xx-idk.  Those nodes carry a
>         unit-address without a matching reg property.
> 
> Each of the two devicetree patches removes a warning that the bare
> conversion leaves behind, so the three belong together:
> 
>   am5729-beagleboneai:  241 -> 237 dtbs_check warnings
>   omap5-igep0050:       188 -> 186 dtbs_check warnings
> 
> with no new warnings in either file.
> 
> Assisted-by: LLM is on all three, which the v1 posting and v3 were
> missing.
> 
> One thing I raised on the thread and should repeat here: Eduard Bostina
> posted the same conversion on 2026-07-30,
> <20260730092245.52034-1-egbostina@gmail.com>.  If his series is the one to
> carry this, say so and I will drop mine.
> 
> Chaosheng Qu (3):
>   dt-bindings: display: ti,tpd12s015: Convert to DT schema
>   ARM: dts: ti/omap: drop the two unused gpios from the igep0050 hdmi
>     encoder
>   ARM: dts: ti/omap: drop unit-address from tpd12s015 endpoints
> 
>  .../bindings/display/ti/ti,tpd12s015.txt           |  44 ------
>  .../bindings/display/ti/ti,tpd12s015.yaml          |  90 ++++++++++++
>  arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts  |   4 +-
>  arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi   |   4 +-
>  arch/arm/boot/dts/ti/omap/omap5-igep0050.dts       |   4 +-
>  5 files changed, 94 insertions(+), 52 deletions(-)

-- 
Regards,

Laurent Pinchart

  parent reply	other threads:[~2026-10-09 11:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-09 10:55 Quchaosheng
2026-10-09 10:55 ` [PATCH 1/3] dt-bindings: display: ti,tpd12s015: Convert to DT schema Quchaosheng
2026-10-09 14:25   ` Conor Dooley
2026-10-09 14:44     ` Laurent Pinchart
2026-10-09 15:42       ` Conor Dooley
2026-10-09 17:34         ` Rob Herring
2026-10-09 15:38   ` Rob Herring
2026-10-09 10:56 ` [PATCH 2/3] ARM: dts: ti/omap: drop the two unused gpios from the igep0050 hdmi encoder Quchaosheng
2026-10-09 10:56 ` [PATCH 3/3] ARM: dts: ti/omap: drop unit-address from tpd12s015 endpoints Quchaosheng
2026-10-09 11:03 ` Laurent Pinchart [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-10-09 10:54 [PATCH 0/3] dt-bindings: display: ti,tpd12s015: convert, and fix what the conversion exposed Quchaosheng
2026-10-09 10:53 Quchaosheng
2026-10-09 10:53 Quchaosheng

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=20261009110306.GB680229@killaraus.ideasonboard.com \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=quchaosheng000406@163.com \
    --cc=robh@kernel.org \
    --cc=tomi.valkeinen@ideasonboard.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®