Hi Stephen,

Please provide your comments on this. I am waiting for the response on this patch series so that I can submit further lan966x code which have dependencies on this gck driver.

Thanks,
Kavya

-----Original Message-----
From: Kavyasree Kotagiri - I30978
Sent: Monday, November 8, 2021 10:42 AM
To: sboyd@kernel.org
Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; linux-
clk@vger.kernel.org; UNGLinuxDriver [off-list ref];
Nicolas Ferre - M43238 [off-list ref]; Eugen Hristev -
M18282 [off-list ref]; Manohar Puri - I30488
[off-list ref]; robh+dt@kernel.org;
mturquette@baylibre.com; Kavyasree Kotagiri - I30978
[off-list ref]
Subject: RE: [PATCH v10 0/3] Add driver for lan966x Generic Clock Controller

Hi Stephen,

Addressed your comments in v10 patch series.
Please let me know when you include my patches into PR.

Thanks,
Kavya

quoted

-----Original Message-----
From: Kavyasree Kotagiri [mailto:kavyasree.kotagiri@microchip.com]
Sent: Wednesday, November 3, 2021 11:50 AM
To: robh+dt@kernel.org; mturquette@baylibre.com; sboyd@kernel.org
Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; linux-
clk@vger.kernel.org; UNGLinuxDriver [off-list ref];
Nicolas Ferre - M43238 [off-list ref]; Eugen Hristev -
M18282 [off-list ref]; Kavyasree Kotagiri - I30978
[off-list ref]; Manohar Puri - I30488
[off-list ref]
Subject: [PATCH v10 0/3] Add driver for lan966x Generic Clock Controller

This patch series adds a device driver for Generic Clock Controller
of lan966x SoC.

v9 -> v10:
- Removed .name from lan966x_gck_pdata struct.
- Removed "_clk" in fw_names like used in bindings

v8 -> v9:
- Added Acked-by to dt-bindings and Documentation file.
- Changed clk_name "timer" to "timer1"
- Updated devm_kzalloc in probe function.

v7 -> v8:
- Defined new constant DIV_MAX.
- Corrected and updated prescaler divider condition check.
- Added Acked-by.

v6 -> v7:
- Added Kconfig and Makefile entires for lan966x clock driver.

v5 -> v6:
- Added Acked-by to dt-bindings file.
- Removed "_clk" in clock-names.
- Added Reviewed-by to Documentation file.

v4 -> v5:
- In v4 dt-bindings, missed adding "clock-names" in required
  properties and example. So, added them.
- Returning proper error - PTR_ERR.
- Removed unused variable "ret" in probe function.

v3 -> v4:
- Updated "clocks" and added "clock-names" in dt-bindings.
- Used clk_parent_data instead of of_clk_get_parent_name().

v2 -> v3:
- Fixed dt_binding_check errors.

v1 -> v2:
- Updated license in dt-bindings.
- Updated example provided for clock controller node.

Kavyasree Kotagiri (3):
  dt-bindings: clock: lan966x: Add binding includes for lan966x SoC
    clock IDs
  dt-bindings: clock: lan966x: Add LAN966X Clock Controller
  clk: lan966x: Add lan966x SoC clock driver

 .../bindings/clock/microchip,lan966x-gck.yaml |  57 +++++
 drivers/clk/Kconfig                           |   7 +
 drivers/clk/Makefile                          |   1 +
 drivers/clk/clk-lan966x.c                     | 240 ++++++++++++++++++
 include/dt-bindings/clock/microchip,lan966x.h |  28 ++
 5 files changed, 333 insertions(+)
 create mode 100644
Documentation/devicetree/bindings/clock/microchip,lan966x-gck.yaml
 create mode 100644 drivers/clk/clk-lan966x.c
 create mode 100644 include/dt-bindings/clock/microchip,lan966x.h

--
2.17.1