From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Fri, 23 Aug 2019 14:48:58 -0700
Subject: [U-Boot] [PATCH V3 1/4] clk: introduce enable_count
In-Reply-To: <20190821135056.27052-1-peng.fan@nxp.com>
References: <20190821135056.27052-1-peng.fan@nxp.com>
Message-ID: <CAPnjgZ2i1Wvq_-4nTWXVuyzCiQ_VJ_1SL2TK90z33mLu=WEdZQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Peng,

On Wed, 21 Aug 2019 at 07:35, Peng Fan <peng.fan@nxp.com> wrote:
>
> As what Linux Kernel 5.3.0 provides when enable/disable clk,
> there is an enable_count in clk_core_disable/enable. Introduce
> enable_count to track the clk enable/disable count when
> clk_enable/disable for CCF. And Initialize enable_count to 0 when
> register the clk.
>
> And clk tree dump with enable_count will be supported, it will
> be easy for us to check the clk status with enable_count
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>
> V3:
>  None
>  CI: https://travis-ci.org/MrVan/u-boot/builds/574753709
> V2:
>  Improve commit log
>  Rename enable_cnt to enable_count following Linux Kernel
>
>  drivers/clk/clk.c            | 1 +
>  drivers/clk/clk_fixed_rate.c | 1 +
>  include/clk.h                | 1 +
>  3 files changed, 3 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>

Do you think it would be worth making this a u8 to save space?

Or perhaps this feature could be optional?

Regards,
Simon