18 Oct, 2016
1 commit
-
The commit 9b4cac33adc7 ("clk: max77686: Migrate to clk_hw based OF and
registration APIs") converted the driver to use the new provider API to
register clocks using clk_hw.But unfortunately, in the conversion it missed to set the num_clks value
which lead to the following error when trying to register a clk provider:[ 1.963782] of_clk_max77686_get: invalid index 0
[ 1.967460] ERROR: could not get clock /rtc@10070000:rtc_src(1)
[ 1.973638] s3c-rtc 10070000.rtc: failed to find rtc source clockFix it by correctly set the max77686_clk_driver_data num_clks member.
Fixes: 9b4cac33adc7 ("clk: max77686: Migrate to clk_hw based OF and registration APIs")
Reported-by: Markus Reichl
Suggested-by: Tobias Jakobi
Signed-off-by: Javier Martinez Canillas
Tested-by: Markus Reichl
Reviewed-by: Chanwoo Choi
Reviewed-by: Krzysztof Kozlowski
Signed-off-by: Stephen Boyd
19 Aug, 2016
1 commit
-
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.Reviewed-by: Javier Martinez Canillas
Tested-by: Javier Martinez Canillas
Cc: Laxman Dewangan
Reviewed-by: Krzysztof Kozlowski
Signed-off-by: Stephen Boyd
Signed-off-by: Stephen Boyd
16 Aug, 2016
2 commits
-
Maxim Max77620 has one 32KHz clock output and the clock HW
IP used on this PMIC is same as what it is there in the MAX77686.Add clock driver support for MAX77620 on the MAX77686 driver.
CC: Krzysztof Kozlowski
CC: Javier Martinez Canillas
Signed-off-by: Laxman Dewangan
Tested-by: Krzysztof Kozlowski
Reviewed-by: Krzysztof Kozlowski
Reviewed-by: Javier Martinez Canillas
Signed-off-by: Stephen Boyd -
The clock IP used on the Maxim PMICs max77686 and max77802 are
same. The configuration of clock register is also same except
the number of clocks.Part of common code utilisation, there is 3 files for these chips
clock driver, one for common and two files for driver registration.Combine both drivers into single file and move common code into
same common file reduces the 2 files and make max77686 and max77802
clock driver in single fine. This driver does not depends on the
parent driver structure. The regmap handle is acquired through
regmap APIs for the register access.This combination of driver helps on adding clock driver for different
Maxim PMICs which has similar clock IP like MAX77620 and MAX20024.Signed-off-by: Laxman Dewangan
CC: Krzysztof Kozlowski
CC: Javier Martinez Canillas
Reviewed-by: Javier Martinez Canillas
Tested-by: Javier Martinez Canillas
Reviewed-by: Krzysztof Kozlowski
Tested-by: Krzysztof Kozlowski
Signed-off-by: Stephen Boyd
16 Mar, 2016
1 commit
-
This flag is a no-op now. Remove usage of the flag.
Reviewed-by: Javier Martinez Canillas
Signed-off-by: Stephen Boyd
17 Jun, 2015
1 commit
-
These files are built off of the tristate COMMON_CLK_MAX77686 and
COMMON_CLK_MAX77802 respectively. They also contains modular function
calls so they should explicitly include module.h to avoid compile
breakage during header shuffles done in the future.Cc: Mike Turquette
Cc: Stephen Boyd
Acked-by: Stephen Boyd
Signed-off-by: Paul Gortmaker
26 Sep, 2014
1 commit
-
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
10 Sep, 2014
2 commits
-
Clocks drivers for Maxim PMIC are very similar so they can
be converted to use the generic Maxim clock driver.Also, while being there use module_platform_driver() helper
macro to eliminate more boilerplate code.Signed-off-by: Javier Martinez Canillas
Reviewed-by: Krzysztof Kozlowski
Signed-off-by: Mike Turquette -
This patch adds a dt-binding include for Maxim 77686
PMIC clock IDs that can be used by both the max77686
clock driver and Device Tree source files.Signed-off-by: Javier Martinez Canillas
Reviewed-by: Krzysztof Kozlowski
Reviewed-by: Mike Turquette
Signed-off-by: Mike Turquette
09 Jan, 2014
7 commits
-
If max77686 chip is instantiated from device tree, it is desirable to
have an OF clock provider to allow device tree based look-up of clocks.
This patch adds OF clock provider registration to the clk-max77686
driver.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
As a prerequisite for further patch adding OF clock provider support to
the driver, this patch changes the driver to store an array of struct
clk * as driver data.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
This patch fixes invalid kfree() and adds missing call to clk_unregister()
in error and remove paths in max77686_clk_probe(). While at it, error
handling is also cleaned up.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
As a preparation for further patches, this patch modifies the clock
registration helper function to return a pointer to the newly registered
clock. No functional change is done to the driver.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
The function can simply return 0, without jumping to a separate label,
which does exactly the same. This patch does not introduce any
functional change, just a clean-up.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
It is usually nice to know frequency of a clock, so this patch adds a
.recalc_rate() callback returning rates of provided clocks.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette -
Changing status of clock gates in max77686 requires i2c transfers, which
can sleep, so this is done in prepare and unprepare callbacks. Due to
this, checking whether whether the clock is ungated must be done
in is_prepared() callback as well, for consistency.Signed-off-by: Tomasz Figa
Acked-by: Kyungmin Park
Signed-off-by: Mike Turquette
27 Dec, 2013
1 commit
-
'break' after 'goto' is redundant. Remove it.
Signed-off-by: Sachin Kamat
Acked-by: Jonghwa Lee
Signed-off-by: Mike Turquette
16 Jan, 2013
2 commits
-
The clk_lookup entry is dropped at remove time by a call to
clkdev_drop(). That function frees the entry, which is also freed by the
driver core as it has been allocated through devm_kzalloc(). This
results in a double free.Use kzalloc() instead of devm_kzalloc() to fix this.
Signed-off-by: Laurent Pinchart
Signed-off-by: Mike Turquette -
container_of() never returns NULL, thus remove the NULL checking for it.
Also rename get_max77686_clk() to to_max77686_clk() for better readability.Signed-off-by: Axel Lin
Signed-off-by: Mike Turquette
12 Jan, 2013
1 commit
-
devm_kzalloc returns NULL on failure.
Signed-off-by: Axel Lin
Signed-off-by: Mike Turquette
29 Nov, 2012
3 commits
-
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.Signed-off-by: Bill Pemberton
Acked-by: Mike Turquette
Acked-by: Mark Brown
Signed-off-by: Greg Kroah-Hartman -
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.Signed-off-by: Bill Pemberton
Acked-by: Mike Turquette
Acked-by: Mark Brown
Signed-off-by: Greg Kroah-Hartman -
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.Signed-off-by: Bill Pemberton
Acked-by: Mike Turquette
Acked-by: Mark Brown
Signed-off-by: Greg Kroah-Hartman
07 Sep, 2012
1 commit
-
This patch supports max77686 mfd's clock driver using common clock frame work.
max77686 has 3 clock ouputs which all are generated from crystal oscillator and
SOC can enable/disable them via I2C bus. All clocks are fixed-rate clock sources
so that it doesn't supply interface for changing clock rate.
Driver uses regmap API to communicate with internal register.Signed-off-by: Jonghwa Lee
Signed-off-by: Mike Turquette
