lwn.git/drivers/clk/clk-wm831x.c, branch v4.17-rc2

clk: make clk_init_data const

2017-11-02T06:25:51+00:00

Make these const as they are only stored in the init field of a clk_hw structure, which is const. Signed-off-by: Bhumika Goyal Signed-off-by: Stephen Boyd

clk: wm831x: fix usleep_range with bad range

2017-01-10T00:06:41+00:00

The delay here is not in atomic context and does not seem critical with respect to precision, but usleep_range(min,max) with min==max results in giving the timer subsystem no room to optimize uncritical delays. Fix this by setting the range to 2000,3000 us. Fixes: commit f05259a6ffa4 ("clk: wm831x: Add initial WM831x clock driver") Signed-off-by: Nicholas Mc Guire Acked-by: Charles Keepax Signed-off-by: Stephen Boyd

clk: clk-wm831x: fix a logic error

2016-12-08T00:24:45+00:00

Fix bug https://bugzilla.kernel.org/show_bug.cgi?id=188561. Function wm831x_clkout_is_prepared() returns "true" when it fails to read CLOCK_CONTROL_1. "true" means the device is already prepared. So return "true" on the read failure seems improper. Signed-off-by: Pan Bian Acked-by: Charles Keepax Fixes: f05259a6ffa4 ("clk: wm831x: Add initial WM831x clock driver") Signed-off-by: Stephen Boyd

clk: wm831x: Migrate to clk_hw based registration APIs

2016-08-25T00:35:53+00:00

Now that we have clk_hw based provider APIs to register clks, we can get rid of struct clk pointers while registering clks in these drivers, allowing us to move closer to a clear split of consumer and provider clk APIs. Cc: Mark Brown Signed-off-by: Stephen Boyd Signed-off-by: Stephen Boyd

clk: wm831x: Remove CLK_IS_ROOT

2016-04-15T23:50:23+00:00

This flag is a no-op now. Remove usage of the flag. Cc: Mark Brown Signed-off-by: Stephen Boyd

clk: wm831x: Remove clk.h include

2015-07-20T17:53:05+00:00

Clock provider drivers generally shouldn't include clk.h because it's the consumer API. Remove the include here because this is a provider driver. Cc: Mark Brown Signed-off-by: Stephen Boyd

clk: Remove .owner field for driver

2014-09-26T00:43:31+00:00

There is no need to init .owner field. Based on the patch from Peter Griffin "mmc: remove .owner field for drivers using module_platform_driver" This patch removes the superflous .owner field for drivers which use the module_platform_driver API, as this is overriden in platform_driver_register anyway." Signed-off-by: Kiran Padwal Signed-off-by: Mike Turquette

clk: wm831x: get rid of the implementation of remove function

2013-10-02T08:23:34+00:00

The remove function implemented for platform driver's remove callback just return 0 as part of its implementation. Remove this APIs and do not pass the valid .remove for platform driver. Signed-off-by: Laxman Dewangan Acked-by: Mark Brown Signed-off-by: Mike Turquette

clk: wm831x: Initialise wm831x pointer on init

2013-08-30T18:57:53+00:00

Otherwise any attempt to interact with the hardware will crash. This is what happens when drivers get written blind. Signed-off-by: Mark Brown Cc: stable@vger.kernel.org Signed-off-by: Mike Turquette

clk: wm831x: Provide is_prepared() rather than is_enabled()

2013-08-30T00:29:01+00:00

Since the driver was written an is_prepared() operation has been made possible. Since the driver uses I2C I/O only prepare operations are provided so move the is_enabled() operation over to is_prepared(). Signed-off-by: Mark Brown Signed-off-by: Mike Turquette