Re: [PATCH v3 2/7] arm: dts: Add the burst and esc clock frequency properties for exynos3250 dts

From: Andi Shyti
Date: Thu Mar 02 2017 - 19:41:05 EST


Hi Hoegeun,

On Thu, Mar 02, 2017 at 07:20:14PM +0900, Hoegeun Kwon wrote:
> The OF graph is not needed because the panel is a child of dsi. So
> added the burst and esc clock frequency properties to the parent (DSI
> node), taking into account the bisectability problem so that remove
> the OF graph from DSI node.

nitpick:

1. bisectability is not a problem (and for bisectability you
should not mention it in the commit log, you would confuse
bisecting people)

2. you should use the imperative form, not "added the burst... "
but "add the burst..."

Same for the other patches.

Andi