linux-kernel - Re: [PATCH v3 4/5] clk: s2mps15: Add support for S2MPS15 clocks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <5630554E.8070301@samsung.com>
Date:	Wed, 28 Oct 2015 10:25:42 +0530
From:	Alim Akhtar <alim.akhtar@...sung.com>
To:	Krzysztof Kozlowski <k.kozlowski@...sung.com>,
	lee.jones@...aro.org, broonie@...nel.org
Cc:	mturquette@...libre.com, linux-samsung-soc@...r.kernel.org,
	linux-clk@...r.kernel.org, rtc-linux@...glegroups.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 4/5] clk: s2mps15: Add support for S2MPS15 clocks



On 10/28/2015 10:04 AM, Krzysztof Kozlowski wrote:
> On 28.10.2015 10:26, Krzysztof Kozlowski wrote:
>> On 26.10.2015 21:51, Alim Akhtar wrote:
>>> S2MPS15 PMIC has three 32k buffered clocks outputs. This patch
>>> adds supports for the same to the s2mps11 clock driver.
>>>
>>> Signed-off-by: Alim Akhtar <alim.akhtar@...sung.com>
>>> ---
>>>   drivers/clk/Kconfig       |    5 +++--
>>>   drivers/clk/clk-s2mps11.c |   24 ++++++++++++++++++++++++
>>>   2 files changed, 27 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
>>> index a1fa61159179..037a314b5d76 100644
>>> --- a/drivers/clk/Kconfig
>>> +++ b/drivers/clk/Kconfig
>>> @@ -120,9 +120,10 @@ config COMMON_CLK_S2MPS11
>>>   	tristate "Clock driver for S2MPS1X/S5M8767 MFD"
>>>   	depends on MFD_SEC_CORE
>>>   	---help---
>>> -	  This driver supports S2MPS11/S2MPS14/S5M8767 crystal oscillator
>>> +	  This driver supports S2MPS1X/S5M8767 crystal oscillator
>>>   	  clock. These multi-function devices have two (S2MPS14) or three
>>> -	  (S2MPS11, S5M8767) fixed-rate oscillators, clocked at 32KHz each.
>>> +	  (S2MPS11/S2MPS13/S2MPS15/S5M8767) fixed-rate oscillators,
>>> +	  clocked at 32KHz each.
>>>
>>>   config CLK_TWL6040
>>>   	tristate "External McPDM functional clock from twl6040"
>>> diff --git a/drivers/clk/clk-s2mps11.c b/drivers/clk/clk-s2mps11.c
>>> index d266299dfdb1..455500dca653 100644
>>> --- a/drivers/clk/clk-s2mps11.c
>>> +++ b/drivers/clk/clk-s2mps11.c
>>> @@ -25,6 +25,7 @@
>>>   #include <linux/mfd/samsung/s2mps11.h>
>>>   #include <linux/mfd/samsung/s2mps13.h>
>>>   #include <linux/mfd/samsung/s2mps14.h>
>>> +#include <linux/mfd/samsung/s2mps15.h>
>>>   #include <linux/mfd/samsung/s5m8767.h>
>>>   #include <linux/mfd/samsung/core.h>
>>>
>>> @@ -148,6 +149,24 @@ static struct clk_init_data s2mps14_clks_init[S2MPS11_CLKS_NUM] = {
>>>   	},
>>>   };
>>>
>>> +static struct clk_init_data s2mps15_clks_init[S2MPS11_CLKS_NUM] = {
>>> +	[S2MPS11_CLK_AP] = {
>>> +		.name = "s2mps15_ap",
>>> +		.ops = &s2mps11_clk_ops,
>>> +		.flags = CLK_IS_ROOT,
>>> +	},
>>> +	[S2MPS11_CLK_CP] = {
>>> +		.name = "s2mps15_cp",
>>> +		.ops = &s2mps11_clk_ops,
>>> +		.flags = CLK_IS_ROOT,
>>> +	},
>>> +	[S2MPS11_CLK_BT] = {
>>> +		.name = "s2mps15_bt",
>>> +		.ops = &s2mps11_clk_ops,
>>> +		.flags = CLK_IS_ROOT,
>>> +	},
>>> +};
>>> +
>>>   static struct device_node *s2mps11_clk_parse_dt(struct platform_device *pdev,
>>>   		struct clk_init_data *clks_init)
>>>   {
>>> @@ -207,6 +226,10 @@ static int s2mps11_clk_probe(struct platform_device *pdev)
>>>   		s2mps11_reg = S2MPS14_REG_RTCCTRL;
>>>   		clks_init = s2mps14_clks_init;
>>>   		break;
>>> +	case S2MPS15X:
>>> +		s2mps11_reg = S2MPS15_REG_RTC_BUF;
>>> +		clks_init = s2mps15_clks_init;
>>
>> Another question (after looking at RTC driver):
>> Is this the same register address as S2MPS14?
>
> I found the answer - this is the same address as in S2MPS13 and S2MPS14:
> 0xc. The S2MPS14 has only two clocks though but S2MPS13 has three.
> Again, as in RTC, you should re-use existing support.
>
> It looks there is no need for this patch at all.
>
Yes, checked that, rtc looks same to s2mps14 and clk looks same as 
s2mps13{ not s2mps11}, because of the RTC_BUF address changes.
Will drop patch 4 and 5 and use the existing names where applicable.
I think I don't have to change the binding or Kconfig changes?


> Best regards,
> Krzysztof
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/