Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
IronPort-SDR: 6tT6uq7p3NIff24VgkP16WaqdGIsI0UdTAuh/gMdZfDijT/7U5DMWVsnPVwERmKC5q20Zii+PT
 MxZb7BWwFdvA==
IronPort-SDR: y/3bTEDA7n9CRAqYwSzGe6+s7KhfcxDilgaVk+4lIyiGhQ3zIMFTuugiuDePCAxAELkUeka0Su
 nhg+bWFtBa7Q==
Subject: Re: [PATCH v7 0/2] clk: intel: Add a new driver for a new clock
 controller IP
To:     sboyd@kernel.org, mturquette@baylibre.com,
        linux-clk@vger.kernel.org
Cc:     robh@kernel.org, mark.rutland@arm.com,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        andriy.shevchenko@intel.com, qi-ming.wu@intel.com,
        yixin.zhu@linux.intel.com, cheol.yong.kim@intel.com
References: <cover.1585022347.git.rahul.tanwar@linux.intel.com>
From:   "Tanwar, Rahul" <rahul.tanwar@linux.intel.com>
Message-ID: <62be98d9-82a1-1062-22e1-4d1d1c008b27@linux.intel.com>
Date:   Mon, 30 Mar 2020 11:29:11 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101
 Thunderbird/68.6.0
MIME-Version: 1.0
In-Reply-To: <cover.1585022347.git.rahul.tanwar@linux.intel.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


Hi Stephen Boyd,

After few rounds of review comments/updates from you & other reviewers,
this patch series seems to be lost :-\. No further review concerns or
acks from past 2 months.

I am wondering if this is due to any outstanding concerns/expectations
or it is just bad luck. Since you are the clk maintainer, may i ask you
to clarify what is it that i am missing. Thanks.

Regards,
Rahul

On 24/3/2020 12:05 pm, Rahul Tanwar wrote:
> Hi,
>
> This series adds clock driver for Clock Generation Unit(CGU) of
> Lightning Mountain(LGM) SoC.
>
> Patch 1 adds bindings document & include file for CGU.
> Patch 2 adds common clock framework based clock driver for CGU.
>
> These patches are baselined upon Linux 5.6-rc1 at below Git link:
> git git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git
>
> v7:
> - Fix a mistake in comments explaining structure fields.
>
> v6:
> - Resolve Kconfig dependencies issues (Randy Dunlap)
>
> v5:
> - Address review concerns - mainly below mentioned. (Stephen Boyd)
> - Improve commit message, add COMPILE_TEST in KConfig dependency.
> - Remove unused header include files, drop unnecessary casts.
> - Switch to using readl_poll_timeout() instead of implementing timeout routine.
> - Avoid using small functions which are called just once. Inline them or
>   remove them.
> - const static --> static const
> - Fix coding style/convention related review concerns.
> - Use __iomem for all IO addresses variables.
> - Consolidate clk_enable & clk_disable ops into a common clk_enable_disable
>   routine to avoid redundant code.
> - Remove unnecessary dev pointers for clk data structures.
> - Redesign code to use new way of specifying clk_parents i.e. use
>   clk_parent_data.fw_name instead of older parent_name strings.
> - Switch from raw_spin_locks() to normal spin_locks() and realign locking.
> - Drop __initconst, __init, __refdata.
> - Reorder patch series - make dt-binding patch as first patch.
> - Add pointer to include file in dt-bindings document.
> - Remove CLK_IS_CRITICAL flag for clks for which IGNORE_UNUSED flag is enough.
>   Add comments for clks which are marked as CRITICAL.
> - Fix $id path in dt-bindings - drop bindings. (Rob Herring).
> - Add Reviewed-by tag from Rob Herring. Thanks Rob.
>
> v4:
> - Add drivers/clk/x86/Kconfig file which got missed in v3 by mistake.
>
> v3:
> - Address review concerns:
>   Add Kconfig entry in x86 folder instead of modifying clk/Kconfig. (Andy Shevchenko)
>   Fix coding style/convention related concerns. (Andy Shevchenko)
>   Improve description, licensing info, rename node name correctly in dt bindings
>   document & remove CLK_NR_CLKS from dt-bindings header file. (Stephen Boyd)
>   Fix a build warning reported by kbuild test robot & Nathan Chancellor
> - Add few new clocks & rename few existing clocks.
> - Add more ops for ddiv & divider clk_ops.
> - Fix few minor bugs.
> - Use CLK_IS_CRITICAL flag for clocks which shall never be disabled.
>
> v2:
> - Move the driver to x86 folder.
> - Remove syscon usage.
> - Remove regmap based access. Use direct readl()/write() instead. Add spinlocks.
> - Change all enum values to capitals.
> - Rename all data structures & functions from intel_* to lgm_*.
> - Remove multiple header files. Keep only one header file.
> - Make probe fail when any of the clk/pll registration fails.
> - Fix few bugs with clk_init_data assignement.
> - Address review concerns for code quality/style/convention.
>
> v1:
> - Initial version.
>
>
> Rahul Tanwar (1):
>   dt-bindings: clk: intel: Add bindings document & header file for CGU
>
> rtanwar (1):
>   clk: intel: Add CGU clock driver for a new SoC
>
>  .../devicetree/bindings/clock/intel,cgu-lgm.yaml   |  44 ++
>  drivers/clk/Kconfig                                |   1 +
>  drivers/clk/x86/Kconfig                            |   8 +
>  drivers/clk/x86/Makefile                           |   1 +
>  drivers/clk/x86/clk-cgu-pll.c                      | 156 +++++
>  drivers/clk/x86/clk-cgu.c                          | 636 +++++++++++++++++++++
>  drivers/clk/x86/clk-cgu.h                          | 335 +++++++++++
>  drivers/clk/x86/clk-lgm.c                          | 492 ++++++++++++++++
>  include/dt-bindings/clock/intel,lgm-clk.h          | 165 ++++++
>  9 files changed, 1838 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/clock/intel,cgu-lgm.yaml
>  create mode 100644 drivers/clk/x86/Kconfig
>  create mode 100644 drivers/clk/x86/clk-cgu-pll.c
>  create mode 100644 drivers/clk/x86/clk-cgu.c
>  create mode 100644 drivers/clk/x86/clk-cgu.h
>  create mode 100644 drivers/clk/x86/clk-lgm.c
>  create mode 100644 include/dt-bindings/clock/intel,lgm-clk.h
>