Re: [[RFC]DPU PATCH 2/4] dt-bindings: drm/bridge: Document sn65dsi86 bridge bindings - spanda-sgV2jX0FEOL9JmXXK+q4OQ

From: spanda-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org
To: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	ryadav-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	linux-arm-msm
	<linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	dri-devel
	<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	Stephen Boyd <swboyd-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	Rob Clark <robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	nganji-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	Sean Paul <seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	abhinavk-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	"Kristian H. Kristensen"
	<hoegsberg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	freedreno
	<freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	Jeykumar Sankaran
	<jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	chandanu-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org
Subject: Re: [[RFC]DPU PATCH 2/4] dt-bindings: drm/bridge: Document sn65dsi86 bridge bindings
Date: Fri, 27 Apr 2018 12:32:05 +0530	[thread overview]
Message-ID: <0522b749900b3a54508abef6bd15252f@codeaurora.org> (raw)
In-Reply-To: <20180427031331.sl62t5ir24dxmtxm@rob-hp-laptop>

On 2018-04-27 08:43, Rob Herring wrote:
> On Wed, Apr 25, 2018 at 08:46:13PM -0400, Rob Clark wrote:
>> On Wed, Apr 25, 2018 at 7:45 PM, Stephen Boyd <swboyd@chromium.org> 
>> wrote:
>> > Quoting Sandeep Panda (2018-04-19 10:56:06)
>> >> Document the bindings used for the sn65dsi86 DSI to eDP bridge.
>> >>
>> >> Changes in v1:
>> >>  - Rephrase the dt-binding descriptions to be more inline with existing
>> >>    bindings (Andrzej Hajda).
>> >>  - Add missing dt-binding that are parsed by corresponding driver
>> >>    (Andrzej Hajda).
>> >>
>> >> Changes in v2:
>> >>  - Removed edp panel specific dt-binding entries. Only keep bridge
>> >>    specific entries (Sean Paul).
>> >>  - Remove custom-modes dt entry since its usage is removed from driver also (Sean Paul).
>> >>  - Remove is-pluggable dt entry since this will not be needed anymore (Sean Paul).
>> >>
>> >> Changes in v3:
>> >>  - Removed irq-gpio dt entry and instead populate is an interrupt
>> >>    property (Rob Herring).
>> >
>> > These changelogs usually go below the triple dash, but maybe drm is
>> > different and wants them?
>> 
>> yeah, drm generally wants them in the commit msg rather than below the
>> triple-dash, although I guess for bindings docs it should follow the
>> rules for that tree.. I usually just fix up these sort of things as I
>> apply patches, but not sure what other maintainers prefer
> 
> Well, these DPU patches aren't targeted for upstream so who cares.

This change is independent of other DPU patches. We are planning to 
upstream these bridge and panel changes.
I will upload the next patchset dropping the DPU tag to avoid any 
confusion.
> 
> Many patch revision changelogs I see are crap with statements like
> "implement changes requested by ??". But in this case, the changelog is
> really good.
> 
> Rob
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno