Re: [PATCH 1/2] clk: samsung: exynos5260: Move struct samsung_cmu_info to init section

2016-08-24 Thread Stephen Boyd
On 08/24, Chanwoo Choi wrote: > On 2016년 08월 24일 13:43, Stephen Boyd wrote: > > On 08/23, Sylwester Nawrocki wrote: > >> So the saving is rather insignificant but the patch doesn't make > >> things worse and I'd say it might be worth applying. > >>

Re: [PATCH 2/2] clk: mvebu: dynamically allocate resources in Armada CP110 system controller

2016-08-24 Thread Stephen Boyd
On 08/23, Marcin Wojtas wrote: > Original commit, which added support for Armada CP110 system controller > used global variables for storing all clock information. It worked > fine for Armada 7k SoC, with single CP110 block. After dual-CP110 Armada 8k > was introduced, the data got overwritten and

Re: [PATCH 13/34] clk: clps711x: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Alexander Sh

Re: [PATCH v2] clk: fixed-factor: add optional dt-binding clock-flags

2016-08-24 Thread Stephen Boyd
On 07/04, Jongsung Kim wrote: > On 2016년 07월 02일 09:20, Stephen Boyd wrote: > > Do you actually have an IC on the board that is doing some fixed > > factor calculation? Or is this a clk driver design where we are > > listing out each piece of an SoC's clk controlle

Re: [PATCH 15/34] clk: efm32gg: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Uwe Kleine-K

Re: [PATCH 1/2] clk: mvebu: set flags in CP110 gate clock

2016-08-24 Thread Stephen Boyd
On 08/23, Marcin Wojtas wrote: > Armada CP110 system controller comprise its own routine responsble > for registering gate clocks. Among others 'flags' field in > struct clk_init_data was not set, using a random values, which > may cause an unpredicted behavior. > > This patch fixes the problem by

Re: [PATCH 14/34] clk: cs2000: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Kuninori Mori

Re: [PATCH 28/34] clk: si5351: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Sebastian

Re: [PATCH 24/34] clk: rk808: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Chris Zhon

Re: [PATCH 29/34] clk: si570: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Guenter

Re: [PATCH 27/34] clk: si514: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Sebastian

Re: [PATCH 26/34] clk: scpi: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Sudeep Holl

Re: [PATCH 33/34] clk: vt8500: Migrate to clk_hw based registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Sebastian

Re: [PATCH 34/34] clk: wm831x: Migrate to clk_hw based registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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 Brow

Re: [PATCH v2] clk: qcom: Migrate to clk_hw based registration and OF APIs

2016-08-24 Thread Stephen Boyd
On 08/16, Stephen Boyd wrote: > Now that we have clk_hw based provider APIs to register clks, we > can get rid of struct clk pointers in this driver, allowing us to > move closer to a clear split of consumer and provider clk APIs. > > Signed-off-by: Stephen Boyd > --- A

Re: [PATCH 06/34] clk: bcm: kona: Migrate to clk_hw based registration and OF APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > Now that we can use clk_hw pointers we don't need to have two > duplicate arrays holding the same mapping of clk index to clk_hw > pointer. Implement a custom clk_hw provider function to map the > OF specifier to the clk_hw instance for it. &g

Re: [PATCH 21/34] clk: nspire: Migrate to clk_hw based OF and registration APIs

2016-08-24 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Daniel Tan

Re: [PATCH 18/34] clk: mb86s7x: Migrate to clk_hw based OF and registration APIs

2016-08-25 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Jassi Bra

Re: [PATCH 22/34] clk: palmas: Migrate to clk_hw based OF and registration APIs

2016-08-25 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Peter Ujfalu

Re: [PATCH 12/34] clk: cdce925: Migrate to clk_hw based OF and provider APIs

2016-08-25 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Mike Looijman

Re: [PATCH 23/34] clk: pwm: Migrate to clk_hw based OF and registration APIs

2016-08-25 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Janusz Uzy

Re: [PATCH 19/34] clk: moxart: Migrate to clk_hw based OF and registration APIs

2016-08-25 Thread Stephen Boyd
On 06/01, Stephen Boyd wrote: > 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: Jonas Jense

Re: [PATCH] clk: qcom: select GDSC for msm8996 gcc and mmcc

2016-08-25 Thread Stephen Boyd
On 08/25, Srinivas Kandagatla wrote: > This patch selects QCOM_GDSC Kconfig for msm8996 GCC and MMCC clock > controllers, as these provide some of the gdscs on the SOC. > > Also selecting this config will make it align with other drivers which > do the same. > > Signed-off-by: Srinivas Kandagatla

Re: [PATCH V5] clk: bcm: Add driver for BCM53573 ILP clock

2016-08-25 Thread Stephen Boyd
On 08/25, Rafał Miłecki wrote: > On 24 August 2016 at 10:47, Stephen Boyd wrote: > > On 08/23, Rafał Miłecki wrote: > >> + } > >> + > >> + /* Disable measurement to save power */ > >> + writel(0x0, pmu + PMU_XTAL_FREQ_RATIO); > >>

Re: [PATCH V5] clk: bcm: Add driver for BCM53573 ILP clock

2016-08-25 Thread Stephen Boyd
On 08/25, Rafał Miłecki wrote: > On 24 August 2016 at 10:47, Stephen Boyd wrote: > > On 08/23, Rafał Miłecki wrote: > >> +static int bcm53573_ilp_enable(struct clk_hw *hw) > >> +{ > >> + struct bcm53573_ilp *ilp = container_of(hw, struct bcm53573_ilp, h

Re: [PATCH 1/2] clk: gcc-msm8996: Fix pcie 2 pipe register offset

2016-08-25 Thread Stephen Boyd
On 08/25, Srinivas Kandagatla wrote: > This patch corrects the register offset for pcie2 pipe clock. > Offset according to datasheet is 0x6e018 instead of 0x6e108. > > Signed-off-by: Srinivas Kandagatla > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Fo

Re: [PATCH 2/2] clk: gcc-msm8996: add missing pcie phy reset lines

2016-08-25 Thread Stephen Boyd
On 08/25, Srinivas Kandagatla wrote: > This patch adds missing 2 PCIE common reset lines. > > Signed-off-by: Srinivas Kandagatla > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: of_clk_add_(hw_)providers multipule times for one node?

2016-08-25 Thread Stephen Boyd
On 08/25, Masahiro Yamada wrote: > > Good. > Could you post it as a patch file? > Sure. I'll write up some commit text. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

[PATCH] clk: Simplify __of_clk_get_hw_from_provider()

2016-08-25 Thread Stephen Boyd
obvious that they're not optional. The assumption is that most providers will implement ->get_hw() so we only fallback to the ->get() function if necessary. This clarifies the intent and removes any possibility of probe defer happening if clk providers are buggy. Reported-by: Masahiro

Re: [PATCH] tty: serial: msm: Add runtime PM and system sleep support

2016-08-25 Thread Stephen Boyd
On 06/17, Pramod Gurav wrote: > @@ -1220,12 +1293,26 @@ static void msm_power(struct uart_port *port, > unsigned int state, > > switch (state) { > case 0: > - clk_prepare_enable(msm_port->clk); > - clk_prepare_enable(msm_port->pclk); > + /* > +

Re: DT connectors, thoughts

2016-08-25 Thread Stephen Boyd
Quoting David Gibson (2016-07-21 21:25:56) > On Thu, Jul 21, 2016 at 02:15:57PM -0500, Rob Herring wrote: > > On Mon, Jul 18, 2016 at 9:20 AM, David Gibson > > wrote: > > > > I understand how you are using i2c alias, but not the intc. It would > > help if the same names were not used in multiple

Re: [PATCH 9/9] clk: imx: add imx7ulp clk driver

2017-06-20 Thread Stephen Boyd
On 06/20, Dong Aisheng wrote: > On Mon, Jun 19, 2017 at 07:01:19PM -0700, Stephen Boyd wrote: > > > > Any reason why it can't be a platform driver? If not, please add > > some comment explaining why. > > > > Timer is using it at early stage. GIC seems

Re: [PATCH] PM / OPP: Add dev_pm_opp_{set|put}_clkname()

2017-06-20 Thread Stephen Boyd
On 06/20, Viresh Kumar wrote: > + */ > +struct opp_table *dev_pm_opp_set_clkname(struct device *dev, const char > *name) > +{ > + struct opp_table *opp_table; > + int ret; > + > + opp_table = dev_pm_opp_get_opp_table(dev); > + if (!opp_table) > + return ERR_PTR(-ENOMEM)

Re: linux-next: build warnings after merge of the clk tree

2017-06-21 Thread Stephen Boyd
te but we had removed it based on > review comments. > > The iproc_pll_xxx() APIs are shared across various clk drivers of iproc SoCs. > > I suggest we bring back __init attribute for all sr_xxx_init() in sr_clk.c ?? > No. We should remove __init from the iproc_pll_clk_setup() f

Re: [PATCH V2] PM / OPP: Add dev_pm_opp_{set|put}_clkname()

2017-06-21 Thread Stephen Boyd
ailable, the OPP core needs to know the > exact name of the clk to use. > > Add a new set of APIs to get that done. > > Tested-by: Rajendra Nayak > Signed-off-by: Viresh Kumar > --- Reviewed-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code

Re: [PATCH 1/5] clk: hisilicon: add usb2 clocks for hi3798cv200 SoC

2017-06-21 Thread Stephen Boyd
On 06/21, Jiancheng Xue wrote: > Add usb2 clocks for hi3798cv200 SoC. > > Signed-off-by: Jiancheng Xue > Reviewed-by: Daniel Thompson > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: uniphier: provide NAND controller clock rate

2017-06-21 Thread Stephen Boyd
On 06/21, Masahiro Yamada wrote: > This allows the NAND driver to get the clock rate via clk_get_rate(). > > Signed-off-by: Masahiro Yamada > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

[PATCH] MAINTAINERS: Add entry for SPMI subsystem

2017-06-21 Thread Stephen Boyd
I have the hardware and I've been reviewing SPMI patches when they come on the list. Add myself as a reviewer in this area and add the linux-arm-msm list because people subscribed there also have the hardware. Cc: Kiran Gunda Cc: Abhijeet Dharmapurikar Signed-off-by: Stephen

Re: [PATCH v2 01/17] clk: ingenic: Use const pointer to clk_ops in struct

2017-06-21 Thread Stephen Boyd
On 06/20, Paul Cercueil wrote: > The CGU common code does not modify the pointed clk_ops structure, so it > should be marked as const. > > Signed-off-by: Paul Cercueil > --- How did you want to merge this series? I can ack clk patches if you like, or apply the clk patches to the clk tree. -- Q

Re: [PATCH] clk: keystone: TI_SCI_PROTOCOL is needed for clk driver

2017-06-21 Thread Stephen Boyd
On 06/20, Arnd Bergmann wrote: > CONFIG_COMPILE_TEST allows building a configuration without > TI_SCI_PROTOCOL, which then fails to link: > > drivers/clk/keystone/sci-clk.o: In function `ti_sci_clk_probe': > sci-clk.c:(.text.ti_sci_clk_probe+0x4c): undefined reference to > `devm_ti_sci_get_handle

Re: [RESEND PATCH v4] clk: stm32h7: Add stm32h743 clock driver

2017-06-21 Thread Stephen Boyd
On 06/07, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > This patch enables clocks for STM32H743 boards. > > Signed-off-by: Gabriel Fernandez > > for MFD changes: > Acked-by: Lee Jones > > for DT-Bindings > Acked-by: Rob Herring > v4: > - rename lock into stm32rcc_lock >

Re: [PATCH] clk: bcm: don't mark iproc_pll_clk_setup as __init

2017-06-21 Thread Stephen Boyd
On 06/21, Arnd Bergmann wrote: > This function is now called from the probe() function of a > platform driver, which may be called later than __init, > which triggers a lot of warnings like: > > WARNING: drivers/clk/built-in.o(.text+0x1a41c): Section mismatch in reference > from the function sr_l

Re: [PATCH v6 2/4] of: make __of_attach_node() static

2017-06-21 Thread Stephen Boyd
On 06/20, frowand.l...@gmail.com wrote: > From: Frank Rowand > > __of_attach_node() is not used outside of drivers/of/dynamic.c. Make > it static and remove it from drivers/of/of_private.h. > > Signed-off-by: Frank Rowand > --- Reviewed-by: Stephen Boyd -- Qualcomm I

Re: [RFC V7 2/2] OPP: Allow "opp-hz" and "opp-microvolt" to contain magic values

2017-11-01 Thread Stephen Boyd
On 11/01, Rob Herring wrote: > On Tue, Oct 31, 2017 at 9:17 PM, Viresh Kumar wrote: > > On 31 October 2017 at 16:02, Rob Herring wrote: > >> Why not a new property for magic values? opp-magic? Don't we want to > >> know when we have magic values? > > > > I have kept a separate property since begi

[PATCH] regulator: qcom_spmi: Include offset when translating voltages

2017-11-01 Thread Stephen Boyd
voltage than what is programmed and the user will be confused. Fix all this by properly offsetting the software selector and hw selector when converting from one number space to another. Fixes: 1b5b19689278 ("regulator: qcom_spmi: Only use selector based regulator ops") Signed-off-

Re: [RFC PATCH] clk: move of_clk_get_parent_count() declaration to

2017-11-01 Thread Stephen Boyd
On 10/29, Masahiro Yamada wrote: > 2017-10-13 8:17 GMT+09:00 Stephen Boyd : > > > USB has a > > similar case, which I think Shawn Guo/Dong Aisheng was trying to > > add an OF based bulk clk_get() API called of_clk_bulk_get() > > for[1]. If this get all clks API

Re: [PATCH v6 1/6] ARM: imx6q: provide documentation for new fsl,pmic-stby-poweroff property

2017-11-01 Thread Stephen Boyd
On 10/27, Oleksij Rempel wrote: > Signed-off-by: Oleksij Rempel > Acked-by: Rob Herring > --- Acked-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH 009/102] clk: sunxi: explicitly request exclusive reset control

2017-11-01 Thread Stephen Boyd
r. Convert all drivers requesting exclusive resets to the > explicit API call so the temporary transition helpers can be removed. > > No functional changes. > > Cc: "Emilio López" > Cc: Michael Turquette > Cc: Stephen Boyd > Cc: Maxime Ripard > Cc: Chen-Yu Tsa

Re: [PATCH v2] clk: sunxi: fix build warning

2017-11-01 Thread Stephen Boyd
On 10/24, Corentin Labbe wrote: > This patch fix the following build warning: > drivers/clk/sunxi/clk-factors.c:279:14: warning: variable 'name' set but not > used [-Wunused-but-set-variable] > > Fixes: 4cbeaebb8af1 ("clk: sunxi: factors: Add unregister function") > > Acked-by: Maxime Ripard >

Re: [PATCH] clk: uniphier: fix clock data for PXs3

2017-11-01 Thread Stephen Boyd
On 10/13, Masahiro Yamada wrote: > Fix reg offsets of USB clocks. > > Fixes: 736de651a836 ("clk: uniphier: add PXs3 clock data") > Signed-off-by: Masahiro Yamada > --- Applied to clk-fixes -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Pr

Re: [PATCH 1/2] CLK: SPEAr: make structure field and function argument as const

2017-11-01 Thread Stephen Boyd
On 10/17, Bhumika Goyal wrote: > Make the masks field of clk_aux structure const as it do not modify the > fields of the aux_clk_masks structure it points to. > > Make the struct aux_clk_masks *aux argument of the function > clk_register_aux as const as the argument is only stored in the masks > f

Re: [PATCH] clk: spear: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: hisilicon: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: make clk_init_data const

2017-11-01 Thread Stephen Boyd
On 08/18, Bhumika Goyal wrote: > 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 > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative

Re: [PATCH] clk: mxs: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: mmp: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: imx: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: sirf: make clk_ops const

2017-11-01 Thread Stephen Boyd
On 08/22, Bhumika Goyal wrote: > Make these const as they are only stored in the const field of a > clk_init_data structure. > > Signed-off-by: Bhumika Goyal > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH 2/2] clk: qcom: common: Migrate to devm_* APIs for resets and clk providers

2017-11-01 Thread Stephen Boyd
On 09/01, Stephen Boyd wrote: > Now that we have devm APIs for the reset controller and of clk hw > provider APIs we can remove the custom code here. > > Signed-off-by: Stephen Boyd > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora

Re: [PATCH 1/2] clk: Add devm_of_clk_add_hw_provider()/del_provider() APIs

2017-11-01 Thread Stephen Boyd
On 09/01, Stephen Boyd wrote: > Sometimes we only have one of_clk_del_provider() call in driver > error and remove paths, because we're missing a > devm_of_clk_add_hw_provider() API. Introduce the API so we can > convert drivers to use this and potentially reduce the amount of

Re: [PATCH v3] clk: at91: utmi: set the mainck rate

2017-11-01 Thread Stephen Boyd
On 09/06, Ludovic Desroches wrote: > By default, it is assumed that the UTMI clock is generated from a 12 MHz > reference clock (MAINCK). If it's not the case, the FREQ field of the > SFR_UTMICKTRIM has to be updated to generate the UTMI clock in the > proper way. > > The UTMI clock has a fixed ra

Re: [PATCH] clk: qcom: Remove unused RCG ops

2017-11-01 Thread Stephen Boyd
On 08/24, Georgi Djakov wrote: > The RCGs ops for shared branches are not used now, so remove it. > > Signed-off-by: Georgi Djakov > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH 2/2] clk: qcom: Modify RCG shared ops to support freq_tbl without XO entry

2017-11-01 Thread Stephen Boyd
On 10/31, Amit Nischal wrote: > There could be some clock sources where there is no entry corresponding > XO in their frequency table, for such sources rcg2_shared_ops would > wrongly configure the RCG registers during enable/disable, which leads > to mismatch between the hardware and software rate

Re: [PATCH 0/2] Add a regmap to Allwinner R40 CCU to export GMAC register

2017-11-02 Thread Stephen Boyd
On 10/09, Icenowy Zheng wrote: > > > 于 2017年10月9日 GMT+08:00 下午3:18:09, Maxime Ripard > 写到: > >On Fri, Oct 06, 2017 at 06:33:31AM +, Icenowy Zheng wrote: > >> In the CCU of the Allwinner R40 SoC, there's a GMAC configuration > >register, > >> which is intended to be accessed by the dwmac-sun

Re: [RFC V7 2/2] OPP: Allow "opp-hz" and "opp-microvolt" to contain magic values

2017-11-02 Thread Stephen Boyd
On 11/02, Viresh Kumar wrote: > On 01-11-17, 14:43, Stephen Boyd wrote: > > On 11/01, Rob Herring wrote: > > > On Tue, Oct 31, 2017 at 9:17 PM, Viresh Kumar > > > wrote: > > > > On 31 October 2017 at 16:02, Rob Herring wrote: > > > >> Why no

Re: [RESEND RFC PATCH 3/7] clk: sunxi: Add CLK_SET_RATE_PARENT flag for H3 HDMI clock

2017-11-02 Thread Stephen Boyd
On 09/20, Jernej Skrabec wrote: > When setting the HDMI clock of H3, the PLL_VIDEO clock needs to be set. > > Add CLK_SET_RATE_PARENT flag for H3 HDMI clock. > > Signed-off-by: Jernej Skrabec > Signed-off-by: Icenowy Zheng > --- Acked-by: Stephen Boyd -- Qualcomm Innov

Re: [PATCH v3 18/22] clk: add support for clocks provided by SCMI

2017-11-02 Thread Stephen Boyd
between the Application Cores(AP) > and the SCP. > > This patch adds support for the clocks provided by SCP using SCMI > protocol. > > Cc: Michael Turquette > Cc: Stephen Boyd > Cc: linux-...@vger.kernel.org > Signed-off-by: Sudeep Holla > --- Acked-by: Stephen Boyd >

Re: [PATCH 1/1] clk: imx6: refine hdmi_isfr's parent to make HDMI work on i.MX6 SoCs w/o VPU

2017-11-02 Thread Stephen Boyd
On 08/01, Sébastien Szymanski wrote: > On i.MX6 SoCs without VPU (in my case MCIMX6D4AVT10AC), the hdmi driver > fails to probe: > > [2.540030] dwhdmi-imx 12.hdmi: Unsupported HDMI controller > (:00:00) > [2.548199] imx-drm display-subsystem: failed to bind 12.hdmi > (ops dw_hd

Re: [PATCH V2 02/10] clk: reparent orphans after critical clocks enabled

2017-11-02 Thread Stephen Boyd
> (start_kernel+0x218/0x394) > [0.00] [] (start_kernel) from [<6000807c>] (0x6000807c) > [0.00] r10: r9:410fc075 r8:6000406a r7:c0e0c930 r6:c0d61a44 > r5:c0e07918 > [ 0.00] r4:c0e78294 > [0.00] ---[ end trace 0

Re: [PATCH V2 01/10] clk: clk-divider: add CLK_DIVIDER_ZERO_GATE clk support

2017-11-02 Thread Stephen Boyd
On 07/13, Dong Aisheng wrote: > diff --git a/drivers/clk/clk-divider.c b/drivers/clk/clk-divider.c > index 9bb472c..55f8c41 100644 > --- a/drivers/clk/clk-divider.c > +++ b/drivers/clk/clk-divider.c > @@ -123,6 +123,9 @@ unsigned long divider_recalc_rate(struct clk_hw *hw, > unsigned long parent_r

Re: [PATCH v6 3/9] clk: mediatek: Add MT2712 clock support

2017-11-02 Thread Stephen Boyd
On 10/23, Weiyi Lu wrote: > Add MT2712 clock support, include topckgen, apmixedsys, > infracfg, pericfg, mcucfg and subsystem clocks. > > Signed-off-by: Weiyi Lu > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Pro

Re: [PATCH v6 1/9] dt-bindings: ARM: Mediatek: Document bindings for MT2712

2017-11-02 Thread Stephen Boyd
On 10/23, Weiyi Lu wrote: > This patch adds the binding documentation for apmixedsys, bdpsys, > imgsys, imgsys, infracfg, mcucfg, mfgcfg, mmsys, pericfg, topckgen, > vdecsys and vencsys for Mediatek MT2712. > > Acked-by: Rob Herring > Signed-off-by: Weiyi Lu > --- Applied to clk-next -- Qualc

Re: [PATCH v6 2/9] clk: mediatek: Add dt-bindings for MT2712 clocks

2017-11-02 Thread Stephen Boyd
On 10/23, Weiyi Lu wrote: > Add MT2712 clock dt-bindings, include topckgen, apmixedsys, > infracfg, pericfg, mcucfg and subsystem clocks. > > Signed-off-by: Weiyi Lu > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative

Re: [PATCH] clk: mediatek: mark mtk_infrasys_init_early __init

2017-11-02 Thread Stephen Boyd
On 09/15, Arnd Bergmann wrote: > On gcc-4.6, we get a harmless link-time warning: > > WARNING: vmlinux.o(.text.unlikely+0x196a0): Section mismatch in reference > from the function mtk_infrasys_init_early() to the function > .init.text:mtk_clk_register_cpumuxes() > The function mtk_infrasys_init_

Re: [PATCH v2 4/4] clk: mediatek: add clocks dt-bindings required header for MT7622 SoC

2017-11-02 Thread Stephen Boyd
On 10/05, sean.w...@mediatek.com wrote: > From: Chen Zhong > > Add the required header for the entire clocks dt-bindings exported > from topckgen, apmixedsys, infracfg, pericfg, ethsys, pciesys, ssusbsys > and audsys which could be found on MT7622 SoC. > > Signed-off-by: Chen Zhong > Signed-off

Re: [PATCH v2 3/4] clk: mediatek: add clock support for MT7622 SoC

2017-11-02 Thread Stephen Boyd
On 10/05, sean.w...@mediatek.com wrote: > From: Sean Wang > > Add all supported clocks exported from every susbystem found on MT7622 SoC > such as topckgen, apmixedsys, infracfg, pericfg , pciessys, ssusbsys, > ethsys and audsys. > > Signed-off-by: Chen Zhong > Signed-off-by: Sean Wang > ---

Re: [PATCH v2 1/4] dt-bindings: clock: mediatek: document clk bindings for MediaTek MT7622 SoC

2017-11-02 Thread Stephen Boyd
On 10/05, sean.w...@mediatek.com wrote: > From: Sean Wang > > This patch adds the binding documentation for apmixedsys, ethsys, hifsys, > infracfg, pericfg, topckgen and audsys for MT7622. > > Signed-off-by: Chen Zhong > Signed-off-by: Sean Wang > Acked-by: Rob Herring > --- Applied to clk-n

Re: [PATCH v2 2/4] clk: mediatek: add the option for determining PLL source clock

2017-11-02 Thread Stephen Boyd
On 10/05, sean.w...@mediatek.com wrote: > From: Chen Zhong > > Since the previous setup always sets the PLL using crystal 26MHz, this > doesn't always happen in every MediaTek platform. So the patch added > flexibility for assigning extra member for determining the PLL source > clock. > > Signed

Re: [PATCH 010/102] clk: tegra: explicitly request exclusive reset control

2017-11-02 Thread Stephen Boyd
r. Convert all drivers requesting exclusive resets to the > explicit API call so the temporary transition helpers can be removed. > > No functional changes. > > Cc: Peter De Schrijver > Cc: Prashant Gaikwad > Cc: Michael Turquette > Cc: Stephen Boyd > Cc: T

Re: [PATCH] clk/tegra: Delete an error message for a failed memory allocation in three functions

2017-11-02 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 27 Sep 2017 18:40:34 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH] clk: sunxi: fix build warning

2017-10-24 Thread Stephen Boyd
On 10/24, Corentin Labbe wrote: > On Tue, Oct 24, 2017 at 01:36:28AM -0700, Stephen Boyd wrote: > > On 10/20, Maxime Ripard wrote: > > > Hi Stephen, Mike, > > > > > > On Thu, Oct 19, 2017 at 09:09:48PM +0200, Corentin Labbe wrote: > > >

Re: [Patch v6 1/7] slimbus: Device management on SLIMbus

2017-10-24 Thread Stephen Boyd
On 10/06, srinivas.kandaga...@linaro.org wrote: > + > +SLIMbus example for Qualcomm's slimbus manager component: > + > + slim@2808 { > + compatible = "qcom,slim-msm"; > + reg = <0x2808 0x2000>, > + interrupts = <0 33 0>; > + clocks = <&lcc

Re: [PATCH 0/3] arm64: remove some unused defconfig options

2017-10-24 Thread Stephen Boyd
On 10/20, Alex Elder wrote: > This series deletes three config options related to USB on Qualcomm > SoCs from the arm64 "defconfig", along with the code that they > enable. The code is no longer needed by any Qualcomm hardware. > > -Alex > > Alex Elder (3)

Re: [RFC 2/4] clk: Add Ingenic X1000 CGU driver

2017-10-25 Thread Stephen Boyd
On 09/27, PrasannaKumar Muralidharan wrote: > Add support for the clocks provided by CGU in Ingenic X1000 SoC. > > Signed-off-by: PrasannaKumar Muralidharan > --- Acked-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation C

Re: [PATCH][clk-next] ARC: clk: fix spelling mistake: "configurarion" -> "configuration"

2017-11-13 Thread Stephen Boyd
On 09/04, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake in dev_dbg message > > Signed-off-by: Colin Ian King > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH] clk: cdce925: remove redundant check for non-null parent_name

2017-11-13 Thread Stephen Boyd
On 09/05, Colin King wrote: > From: Colin Ian King > > The check for a non-null parent_name is always true as there is an > earlier check for a null parent_name that returns -ENODEV. Remove this > redundant check and always set init.num_parents to 1. > > Detected by CoverityScan CID#1309477 ("Lo

Re: [PATCH 2/2] clk-mux: Improve a size determination in clk_hw_register_mux_table()

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 17:30:06 +0200 > > Replace the specification of a data structure by a pointer dereference > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Lin

Re: [PATCH] [media] clk-kona-setup: Delete two error messages for a failed memory allocation in parent_process()

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 16:54:20 +0200 > > Omit extra messages for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-next

Re: [PATCH 1/2] clk-mux: Delete an error message for a failed memory allocation in clk_hw_register_mux_table()

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 17:23:04 +0200 > > Omit an extra message for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH 4/4] clk-u300: Fix a typo in two comment lines

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 19:57:50 +0200 > > Add a missing character in these descriptions. > > Signed-off-by: Markus Elfring > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Found

Re: [PATCH 3/4] clk-u300: Add some spaces for better code readability

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 19:46:51 +0200 > > Use space characters at some source code places according to > the Linux coding style convention. > > Signed-off-by: Markus Elfring > --- Applied to clk-next -- Qualcomm Innovation Center, I

Re: [PATCH 1/4] clk-u300: Delete an error message for a failed memory allocation in two functions

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 19:30:46 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH 2/2] clk-xgene: Adjust six checks for null pointers

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 21:27:52 +0200 > MIME-Version: 1.0 > Content-Type: text/plain; charset=UTF-8 > Content-Transfer-Encoding: 8bit > > The script “checkpatch.pl” pointed information out like the following. > > Comparison to NULL coul

Re: [PATCH 2/4] clk-u300: Improve a size determination in two functions

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 19:33:02 +0200 > > Replace the specification of data structures by pointer dereferences > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Linux

Re: [PATCH 1/3] clk/mmp: Delete an error message for a failed memory allocation in three functions

2017-11-13 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 22:25:38 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH] clk/spear: Delete five error messages for a failed memory allocation

2017-11-13 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 27 Sep 2017 18:00:21 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH 1/2] clk-xgene: Delete an error message for a failed memory allocation in two functions

2017-11-13 Thread Stephen Boyd
On 09/26, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Sep 2017 21:19:19 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH] clk/ti: Delete an error message for a failed memory allocation in two functions

2017-11-13 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 27 Sep 2017 19:14:01 +0200 > > Omit extra messages for a memory allocation failure in these functions. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- Applied to clk-ne

Re: [PATCH 2/2] clk/ux500: Improve a size determination in three functions

2017-11-13 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 27 Sep 2017 20:30:53 +0200 > > Replace the specification of data structures by pointer dereferences > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Linux

Re: [PATCH 2/2] clk/versatile: Improve a size determination in icst_clk_setup()

2017-11-13 Thread Stephen Boyd
On 09/27, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 27 Sep 2017 21:24:43 +0200 > > Replace the specification of a data structure by a pointer dereference > as the parameter for the operator "sizeof" to make the corresponding size > determination a bit safer according to the Lin

<    5   6   7   8   9   10   11   12   13   14   >