From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Agner Subject: Re: [PATCH] ARM: dts: imx7d: fix LCDIF clock assignment Date: Sun, 04 Dec 2016 17:26:58 -0800 Message-ID: <7e4829f484f6c4425fc9d01bea1a094f@agner.ch> References: <20161123004204.10851-1-stefan@agner.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Shawn Guo Cc: Mark Rutland , devicetree@vger.kernel.org, Fabio Estevam , linux-kernel , robh+dt@kernel.org, Peter Chen , Sascha Hauer , Fabio Estevam , Liu Ying , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hi Shawn On 2016-11-23 15:02, Fabio Estevam wrote: > On Tue, Nov 22, 2016 at 10:42 PM, Stefan Agner wrote: >> The eLCDIF IP of the i.MX 7 SoC knows multiple clocks and lists them >> separately: >> >> Clock Clock Root Description >> apb_clk MAIN_AXI_CLK_ROOT AXI clock >> pix_clk LCDIF_PIXEL_CLK_ROOT Pixel clock >> ipg_clk_s MAIN_AXI_CLK_ROOT Peripheral access clock >> >> All of them are switched by a single gate, which is part of the >> IMX7D_LCDIF_PIXEL_ROOT_CLK clock. Hence using that clock also for >> the AXI bus clock (clock-name "axi") makes sure the gate gets >> enabled when accessing registers. >> >> There seem to be no separate AXI display clock, and the clock is >> optional. Hence remove the dummy clock. >> >> This fixes kernel freezes when starting the X-Server (which >> disables/re-enables the display controller). >> >> Signed-off-by: Stefan Agner > > Reviewed-by: Fabio Estevam Since this fixes a kernel freeze, is there a chance to get this still in 4.9? -- Stefan