[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20201215004420.11342-1-liambeguin@gmail.com>
Date: Mon, 14 Dec 2020 19:44:19 -0500
From: Liam Beguin <liambeguin@...il.com>
To: liambeguin@...il.com, mturquette@...libre.com, sboyd@...nel.org
Cc: linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: [RFC PATCH v1 0/1] add support for the LMK04832
From: Liam Beguin <lvb@...hos.com>
Hi,
The LMK04832 is an ultra-high performance clock conditioner with JEDEC
JESD204B support and is also pin compatible with the LMK0482x family of
devices.
This driver adds minimal support to configure the LMK04832 clocks using
the clock framework.
At the moment, this doesn't fully support JESD204B subclass 1.
I'm working on adding proper SYSREF support and will update this series
as soon as it's ready.
At this stage, I was hoping for some general feedback on the work done
so far and advice on a few points:
- Currently, nothing sets the VCO frequency. In a local version,
I used a devicetree property and clk_set_rate(lmk->vco.clk, vco_rate).
Should this be set using the parent_rate from inside the
lmk04832_sclk_round_rate() and lmk04832_dclk_round_rate() functions.
How would this affect VCO sub clocks?
- Should round_rate error out when the calculated rate is too far from
the desired rate? Other drivers don't seem to do that.
- Since this is device is compatible with the LMK0482x family, should
the driver be renamed to lmk048xx?
Thanks for your time,
Liam
Liam Beguin (1):
clk: add initial support for the lmk04832
.../devicetree/bindings/clock/ti,lmk04832.txt | 90 ++
drivers/clk/Kconfig | 7 +
drivers/clk/Makefile | 1 +
drivers/clk/clk-lmk04832.c | 1357 +++++++++++++++++
4 files changed, 1455 insertions(+)
create mode 100644 Documentation/devicetree/bindings/clock/ti,lmk04832.txt
create mode 100644 drivers/clk/clk-lmk04832.c
base-commit: 2c85ebc57b3e1817b6ce1a6b703928e113a90442
--
2.27.0
Powered by blists - more mailing lists