All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Cc: David Airlie <airlied@linux.ie>, Rob Herring <robh+dt@kernel.org>,
	dri-devel@lists.freedesktop.org,
	linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
	Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH] dt-bindings: display: renesas: du: document R8A77980 bindings
Date: Tue, 05 Jun 2018 22:16:20 +0300	[thread overview]
Message-ID: <3434502.pBJZTsWR88@avalon> (raw)
In-Reply-To: <df4d4207-5343-36c0-7ca8-d431999a3f31@cogentembedded.com>

Hi Sergei,

On Tuesday, 5 June 2018 21:57:47 EEST Sergei Shtylyov wrote:
> On 06/05/2018 01:09 PM, Laurent Pinchart wrote:
> >> Document the R-Car V3H (R8A77980) SoC in the R-Car DU bindings; the DU
> >> hardware seems the same as in the R-Car V3M (R8A77970).
> > 
> > How about "the DU hardware has the same topology as in the R-Car V3M
> > (R8A77970)" ? "seems" sounds like we're very unsure :-)
> 
>    That's probably better, indeed.
> 
> >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >> 
> >> ---
> >> The patch is against the 'drm-next' branch of David Airlie's 'linux.git'
> >> repo.
> > 
> > Then you might want to switch to git://anongit.freedesktop.org/drm/drm :-)
> 
> No, the corresponding MAINTAINERS records don't include
> drivers/gpu/drm/rcar-du/ or worse yet, the DU bindings. :-)

My point is that Dave's tree has moved.

> Seriously, I thought you've done a patch to remove your dead 'fbdev' repo
> months ago and IO'm still seeing it listed... :-/

I'll address that now.

> > Apart from that,
> > 
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > 
> > If you agree with the small change to the commit message I'll fix the
> > conflict locally, there's no need to resubmit.
> 
> I do agree, except I don't know what conflict you mean.
> 
> [...]

I mean the conflict with

commit dc8142901befabea974393d49b803f131243feb4
Author: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Date:   Thu Apr 26 17:53:31 2018 +0100

    dt-bindings: display: renesas: du: Document the r8a77965 bindings

that is present in Dave's new tree.

-- 
Regards,

Laurent Pinchart

WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, David Airlie <airlied@linux.ie>,
	dri-devel@lists.freedesktop.org,
	linux-renesas-soc@vger.kernel.org,
	Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH] dt-bindings: display: renesas: du: document R8A77980 bindings
Date: Tue, 05 Jun 2018 22:16:20 +0300	[thread overview]
Message-ID: <3434502.pBJZTsWR88@avalon> (raw)
In-Reply-To: <df4d4207-5343-36c0-7ca8-d431999a3f31@cogentembedded.com>

Hi Sergei,

On Tuesday, 5 June 2018 21:57:47 EEST Sergei Shtylyov wrote:
> On 06/05/2018 01:09 PM, Laurent Pinchart wrote:
> >> Document the R-Car V3H (R8A77980) SoC in the R-Car DU bindings; the DU
> >> hardware seems the same as in the R-Car V3M (R8A77970).
> > 
> > How about "the DU hardware has the same topology as in the R-Car V3M
> > (R8A77970)" ? "seems" sounds like we're very unsure :-)
> 
>    That's probably better, indeed.
> 
> >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >> 
> >> ---
> >> The patch is against the 'drm-next' branch of David Airlie's 'linux.git'
> >> repo.
> > 
> > Then you might want to switch to git://anongit.freedesktop.org/drm/drm :-)
> 
> No, the corresponding MAINTAINERS records don't include
> drivers/gpu/drm/rcar-du/ or worse yet, the DU bindings. :-)

My point is that Dave's tree has moved.

> Seriously, I thought you've done a patch to remove your dead 'fbdev' repo
> months ago and IO'm still seeing it listed... :-/

I'll address that now.

> > Apart from that,
> > 
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > 
> > If you agree with the small change to the commit message I'll fix the
> > conflict locally, there's no need to resubmit.
> 
> I do agree, except I don't know what conflict you mean.
> 
> [...]

I mean the conflict with

commit dc8142901befabea974393d49b803f131243feb4
Author: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Date:   Thu Apr 26 17:53:31 2018 +0100

    dt-bindings: display: renesas: du: Document the r8a77965 bindings

that is present in Dave's new tree.

-- 
Regards,

Laurent Pinchart



_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2018-06-05 19:16 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-04 19:04 [PATCH] dt-bindings: display: renesas: du: document R8A77980 bindings Sergei Shtylyov
2018-06-04 19:04 ` Sergei Shtylyov
2018-06-05  8:09 ` Simon Horman
2018-06-05  8:09   ` Simon Horman
2018-06-05 10:09 ` Laurent Pinchart
2018-06-05 10:09   ` Laurent Pinchart
2018-06-05 18:57   ` Sergei Shtylyov
2018-06-05 18:57     ` Sergei Shtylyov
2018-06-05 19:16     ` Laurent Pinchart [this message]
2018-06-05 19:16       ` Laurent Pinchart
2018-06-05 19:49       ` Sergei Shtylyov
2018-06-05 19:49         ` Sergei Shtylyov
2018-06-05 20:24         ` Laurent Pinchart
2018-06-05 20:24           ` Laurent Pinchart
2018-06-06  7:04           ` Geert Uytterhoeven
2018-06-06  7:04             ` Geert Uytterhoeven
2018-06-12 21:02 ` Rob Herring
2018-06-12 21:02   ` Rob Herring

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=3434502.pBJZTsWR88@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=airlied@linux.ie \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=sergei.shtylyov@cogentembedded.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.