Hi Tom,
On 05/08/2018 01:39, Tom Rini wrote:
On Thu, Jul 26, 2018 at 03:54:04PM +0200, Neil Armstrong wrote:
This patch adds support for :
- Rate calculation through muxes and generic dividers
- Basic gate setting propagation
- Reparenting for muxes
- Clock rate setting through generic dividers without reparenting
Support is only added to the Composite VPU and VAPB clocks in order to support the Video Processing Unit Power Domain clock setup.
Signed-off-by: Neil Armstrong narmstrong@baylibre.com Reviewed-by: Simon Glass sjg@chromium.org
I get: Building current source for 1 boards (1 thread, 16 jobs per thread) aarch64: + khadas-vim +(khadas-vim) In file included from include/linux/bug.h:7:0, +(khadas-vim) from include/common.h:25, +(khadas-vim) from drivers/clk/clk_meson.c:8: +(khadas-vim) debug("%s: %sabling %d\n", __func__, on ? "en" : "dis", id); +(khadas-vim) ^ +(khadas-vim) include/linux/printk.h:37:21: note: in definition of macro 'pr_fmt' +(khadas-vim) #define pr_fmt(fmt) fmt +(khadas-vim) ^~~ +(khadas-vim) include/log.h:141:2: note: in expansion of macro 'debug_cond' +(khadas-vim) debug_cond(_DEBUG, fmt, ##args) +(khadas-vim) ^~~~~~~~~~ +(khadas-vim) drivers/clk/clk_meson.c:198:2: note: in expansion of macro 'debug' +(khadas-vim) ^~~~~ +(khadas-vim) debug("%s: really %sabling %d\n", __func__, on ? "en" : "dis", id); +(khadas-vim) drivers/clk/clk_meson.c:218:2: note: in expansion of macro 'debug' +(khadas-vim) arch/arm/dts/meson-gxl-s905x-khadas-vim.dtb: Warning (avoid_unnecessary_addr_size): /gpio-keys-polled: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property w+(khadas-vim) drivers/clk/clk_meson.c: In function 'meson_set_gate_by_id': w+(khadas-vim) drivers/clk/clk_meson.c:198:8: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] w+(khadas-vim) drivers/clk/clk_meson.c:218:8: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
Thanks for pointing me these, I missed these...
I'll repost a fixed version ASAP.
Neil