kHi Lukasz, On Thu, 16 May 2019 at 16:11, Lukasz Majewski <lu...@denx.de> wrote: > > This patch series brings the files from Linux kernel to provide clocks support > as it is used on the Linux kernel with common clock framework [CCF] setup. > > This series also fixes several problems with current clocks and provides > sandbox tests for functions addded to clk-uclass.c file. > > Repository: > https://github.com/lmajewski/u-boot-dfu/commits/CCF-v4 > > > Changes in v4: > - New patch > - None > - None > - None > - New patch > - None > - Port some more Linux code to facilitate imx8 code porting (most notably > flags) > - Explicitly use container_of() based macro to provide struct clk in various > places (e.g. gate2, mux, etc) > Following patches has been squashed: > http://patchwork.ozlabs.org/patch/1093141/ > http://patchwork.ozlabs.org/patch/1093142/ > http://patchwork.ozlabs.org/patch/1093146/ > - New patch > > Changes in v3: > - New patch > - The rate information is now cached into struct clk field > - The clk_get_parent() is used to get pointer to the parent struct clk > - Replace -ENODEV with -ENOENT > - Use **clkp instead of **c > - New patch > - New patch > > Lukasz Majewski (13): > clk: doc: Add documentation entry for Common Clock Framework [CCF] > (i.MX) > dm: Fix documentation entry as there is no UCLASS_CLOCK uclass > clk: Remove clock ID check in .get_rate() of clk_fixed_* > clk: Extend struct clk to provide information regarding clock rate > clk: Extend struct clk to provide clock type agnostic flags > clk: Provide struct clk for fixed rate clock (clk_fixed_rate.c) > dm: clk: Define clk_get_parent() for clk operations > dm: clk: Define clk_get_parent_rate() for clk operations > dm: clk: Define clk_get_by_id() for clk operations > clk: test: Provide unit test for clk_get_by_id() method > clk: test: Provide unit test for clk_get_parent_rate() method > clk: Port Linux common clock framework [CCF] for imx6q to U-boot (tag: > 5.0-rc3) > dm: clk: Extend clk_get_parent_rate() to support CLK_GET_RATE_NOCACHE > flag > > arch/sandbox/include/asm/clk.h | 16 ++++ > doc/imx/clk/ccf.txt | 83 +++++++++++++++++++ > drivers/clk/Kconfig | 14 ++++ > drivers/clk/Makefile | 2 + > drivers/clk/clk-divider.c | 148 ++++++++++++++++++++++++++++++++++ > drivers/clk/clk-fixed-factor.c | 87 ++++++++++++++++++++ > drivers/clk/clk-mux.c | 164 +++++++++++++++++++++++++++++++++++++ > drivers/clk/clk-uclass.c | 60 ++++++++++++++ > drivers/clk/clk.c | 56 +++++++++++++ > drivers/clk/clk_fixed_factor.c | 3 - > drivers/clk/clk_fixed_rate.c | 8 +- > drivers/clk/clk_sandbox_test.c | 49 +++++++++++ > drivers/clk/imx/Kconfig | 9 +++ > drivers/clk/imx/Makefile | 2 + > drivers/clk/imx/clk-gate2.c | 113 ++++++++++++++++++++++++++ > drivers/clk/imx/clk-imx6q.c | 179 > +++++++++++++++++++++++++++++++++++++++++ > drivers/clk/imx/clk-pfd.c | 91 +++++++++++++++++++++ > drivers/clk/imx/clk-pllv3.c | 83 +++++++++++++++++++ > drivers/clk/imx/clk.h | 75 +++++++++++++++++ > include/clk.h | 37 ++++++++- > include/linux/clk-provider.h | 115 ++++++++++++++++++++++++++ > test/dm/clk.c | 4 +- > 22 files changed, 1390 insertions(+), 8 deletions(-) > create mode 100644 doc/imx/clk/ccf.txt > create mode 100644 drivers/clk/clk-divider.c > create mode 100644 drivers/clk/clk-fixed-factor.c > create mode 100644 drivers/clk/clk-mux.c > create mode 100644 drivers/clk/clk.c > create mode 100644 drivers/clk/imx/clk-gate2.c > create mode 100644 drivers/clk/imx/clk-imx6q.c > create mode 100644 drivers/clk/imx/clk-pfd.c > create mode 100644 drivers/clk/imx/clk-pllv3.c > create mode 100644 drivers/clk/imx/clk.h > create mode 100644 include/linux/clk-provider.h
I don't see any new tests here other a trivial change in test/dm/clk.c When adding new functionality to the generic clock code, tests should be added to cover this code. The sandbox clock driver should be enhanced as needed. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot