All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	dri-devel@lists.freedesktop.org, David Airlie <airlied@gmail.com>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-kernel@vger.kernel.org, Sam Ravnborg <sam@ravnborg.org>,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	Del Regno <angelogioacchino.delregno@somainline.org>,
	devicetree@vger.kernel.org, Daniel Vetter <daniel@ffwll.ch>,
	Jianhua Lu <lujianhua000@gmail.com>,
	Neil Armstrong <neil.armstrong@linaro.org>
Subject: Re: [PATCH v3 2/3] dt-bindings: display: novatek,nt35950: define ports
Date: Wed, 23 Aug 2023 07:48:17 -0500	[thread overview]
Message-ID: <20230823124817.GA2034925-robh@kernel.org> (raw)
In-Reply-To: <32667691-ec7f-0dd3-b3ba-fdcd40f1fbca@linaro.org>

On Wed, Aug 23, 2023 at 11:53:56AM +0200, Krzysztof Kozlowski wrote:
> On 23/08/2023 11:08, Rob Herring wrote:
> > 
> > On Wed, 23 Aug 2023 10:14:59 +0200, Krzysztof Kozlowski wrote:
> >> The panel-common schema does not define what "ports" property is, so
> >> bring the definition by referencing the panel-common-dual.yaml. Panels
> >> can be single- or dual-link, thus require only one port@0.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>
> >> ---
> >>
> >> Changes since v2:
> >> 1. Use panel-common-dual
> >>
> >> Changes since v1:
> >> 1. Rework to add ports to device schema, not to panel-common.
> >> ---
> >>  .../devicetree/bindings/display/panel/novatek,nt35950.yaml     | 3 ++-
> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> > 
> > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> > on your patch (DT_CHECKER_FLAGS is new in v5.13):
> > 
> > yamllint warnings/errors:
> 
> Previous patch seems to be missing in Patchwork, thus this error.
> 
> https://patchwork.ozlabs.org/project/devicetree-bindings/list/?submitter=83726&archive=both&state=*

Must have been some delay on that one as it is there now.

Rob

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: devicetree@vger.kernel.org, Conor Dooley <conor+dt@kernel.org>,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Thierry Reding <thierry.reding@gmail.com>,
	Jianhua Lu <lujianhua000@gmail.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Del Regno <angelogioacchino.delregno@somainline.org>
Subject: Re: [PATCH v3 2/3] dt-bindings: display: novatek,nt35950: define ports
Date: Wed, 23 Aug 2023 07:48:17 -0500	[thread overview]
Message-ID: <20230823124817.GA2034925-robh@kernel.org> (raw)
In-Reply-To: <32667691-ec7f-0dd3-b3ba-fdcd40f1fbca@linaro.org>

On Wed, Aug 23, 2023 at 11:53:56AM +0200, Krzysztof Kozlowski wrote:
> On 23/08/2023 11:08, Rob Herring wrote:
> > 
> > On Wed, 23 Aug 2023 10:14:59 +0200, Krzysztof Kozlowski wrote:
> >> The panel-common schema does not define what "ports" property is, so
> >> bring the definition by referencing the panel-common-dual.yaml. Panels
> >> can be single- or dual-link, thus require only one port@0.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>
> >> ---
> >>
> >> Changes since v2:
> >> 1. Use panel-common-dual
> >>
> >> Changes since v1:
> >> 1. Rework to add ports to device schema, not to panel-common.
> >> ---
> >>  .../devicetree/bindings/display/panel/novatek,nt35950.yaml     | 3 ++-
> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> > 
> > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> > on your patch (DT_CHECKER_FLAGS is new in v5.13):
> > 
> > yamllint warnings/errors:
> 
> Previous patch seems to be missing in Patchwork, thus this error.
> 
> https://patchwork.ozlabs.org/project/devicetree-bindings/list/?submitter=83726&archive=both&state=*

Must have been some delay on that one as it is there now.

Rob

  reply	other threads:[~2023-08-23 12:48 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-23  8:14 [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema Krzysztof Kozlowski
2023-08-23  8:14 ` [PATCH v3 2/3] dt-bindings: display: novatek,nt35950: define ports Krzysztof Kozlowski
2023-08-23  9:08   ` Rob Herring
2023-08-23  9:08     ` Rob Herring
2023-08-23  9:53     ` Krzysztof Kozlowski
2023-08-23  9:53       ` Krzysztof Kozlowski
2023-08-23 12:48       ` Rob Herring [this message]
2023-08-23 12:48         ` Rob Herring
2023-08-24 17:20   ` Rob Herring
2023-08-24 17:20     ` Rob Herring
2023-08-23  8:15 ` [PATCH v3 3/3] dt-bindings: display: novatek,nt36523: " Krzysztof Kozlowski
2023-08-24 17:21   ` Rob Herring
2023-08-24 17:21     ` Rob Herring
2023-08-23  8:34 ` [PATCH v3 1/3] dt-bindings: display: panel: add common dual-display schema Laurent Pinchart
2023-08-23  8:34   ` Laurent Pinchart
2023-08-23  9:35   ` Krzysztof Kozlowski
2023-08-23  9:35     ` Krzysztof Kozlowski
2023-08-23 10:07     ` Laurent Pinchart
2023-08-23 10:07       ` Laurent Pinchart

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=20230823124817.GA2034925-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=airlied@gmail.com \
    --cc=angelogioacchino.delregno@somainline.org \
    --cc=conor+dt@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lujianhua000@gmail.com \
    --cc=neil.armstrong@linaro.org \
    --cc=sam@ravnborg.org \
    --cc=thierry.reding@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 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.