Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752403AbbDAJQM (ORCPT ); Wed, 1 Apr 2015 05:16:12 -0400 Received: from mail-yh0-f52.google.com ([209.85.213.52]:32815 "EHLO mail-yh0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750793AbbDAJQJ (ORCPT ); Wed, 1 Apr 2015 05:16:09 -0400 MIME-Version: 1.0 In-Reply-To: <7h1tk47awc.fsf@deeprootsystems.com> References: <1427730803-28635-1-git-send-email-javier.martinez@collabora.co.uk> <1427730803-28635-3-git-send-email-javier.martinez@collabora.co.uk> <551976F1.1000605@collabora.co.uk> <7hiodg98z7.fsf@deeprootsystems.com> <7h1tk47awc.fsf@deeprootsystems.com> Date: Wed, 1 Apr 2015 11:16:08 +0200 X-Google-Sender-Auth: GKM1mwlbuIbHvddcfodUIBu-AMs Message-ID: Subject: Re: [RFC PATCH v3 2/2] clk: exynos5420: Make sure MDMA0 clock is enabled during suspend From: Krzysztof Kozlowski To: Kevin Hilman Cc: Abhilash Kesavan , Krzysztof Kozlowski , "linux-samsung-soc@vger.kernel.org" , Mike Turquette , linux-kernel , Tyler Baker , Stephen Boyd , Tomasz Figa , Doug Anderson , Chanwoo Choi , Kukjin Kim , Sylwester Nawrocki , Olof Johansson , Javier Martinez Canillas , linux-arm-kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2410 Lines: 60 2015-04-01 6:03 GMT+02:00 Kevin Hilman : > Abhilash Kesavan writes: > >> On Wed, Apr 1, 2015 at 2:32 AM, Kevin Hilman wrote: >>> Javier Martinez Canillas writes: >>> >>> [...] >>> >>>> Unfortunately I don't fully understand why this clock needs to be >>>> enabled. It would be good if someone at Samsung can explain in more >>>> detail what the real problem really is. >>> >>> +1 >>> >>> Maybe Abhilash can shed some light here? >>> >>> We really should know *why* this is needed because having the fix in the >>> clock driver just doesn't seem right. It seems like the DMA driver >>> should be managing this clock. >> >> I think my last mail might not have reached you (was accidentally sent >> as html). > > Yeah, I saw it a bit later in Javier's reply. Thanks for doing the > research and reporting back. > >> We are gating the aclk266_g2d clock without checking the >> CG_STATUS0 register bits as specified in the UM. It looks like we need >> to keep several clocks alive or gate them only after checking the >> CG_STATUSx register bits. > > I dont' know much about this clock hardware, but to me it sounds like a > clock driver bug. The suspend fix Javier is proposing would fix it, but > to me it sounds like the clock driver needs to actually start checking > these CG_STATUSx bits before gating clocks. > > Otherwise, we might fix this current bug but a similar one will come and > bite us another day. Actually it looks kind a similar to issue with adma/mau_epll clocks: https://lkml.org/lkml/2014/12/5/291 In that case runtime PM for pl330 caused adma clock to be gated. This lead to gating its parent clock - mau_epll. The clock hierarchy is strange for maudio domain. Basically mau_epll is needed to access maudio clocks however these clocks are not children of mau_epll. I think we should not enable the mdma0 but instead we should find the proper parent which needs to be enabled. Anyway it is kind of annoying (or funny if one has sense of black humour) that two issues are exposed by runtime PM for pl330 driver. Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/