Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-07-30 Thread Vikas Sajjan
Hi Mike,

On Tue, Jul 23, 2013 at 6:56 AM, Mike Turquette mturque...@linaro.org wrote:
 Quoting Vikas Sajjan (2013-07-21 23:15:27)
 Hi Kukjin / Mike,

 Can you apply this series

 Let me apply this to see if the dependencies are taken care of now. If
 so I'll just take it through the clk tree.


Any updates on this series.
I could NOT see them at
http://git.linaro.org/gitweb?p=people/mturquette/linux.git;a=shortlog;h=refs/heads/clk-next


 Regards,
 Mike


 On Tue, Jul 16, 2013 at 7:10 PM, Yadwinder Singh Brar
 yadi.bra...@gmail.com wrote:
  Hi Kukjin / Mike,
 
  On Tue, Jun 25, 2013 at 7:40 AM, Kukjin Kim kgene@samsung.com wrote:
  Mike Turquette wrote:
 
  Quoting Kukjin Kim (2013-06-24 08:02:39)
   On 06/22/13 01:31, Mike Turquette wrote:
   
Acked-by: Mike Turquettemturque...@linaro.org
   
Or I can take this through the clk tree. Either way is fine.
   
   Mike, please go ahead taking this series into the clk tree for 3.11.
 
  Actually it might be better for you to take it. Patch #5 relies on
  drivers/clk/samsung/clk-exynos5420.c which only exists in the samsung
  tree.  It was introduced by commit 1609027f, clk: exynos5420: register
  clocks using common clock framework.
 
  Yes, you're right. It depends on soc-exynos5420 stuff in samsung tree.
 
  And these patches need to be applied with some fuzz to the clk tree due
  to other changes which are not in my tree and only present in yours.
 
  The best two options are either for you to take this series or it can be
  rebased on top of clk-next, after which I'll apply it.
 
  Let me try to send this series via samsung tree for v3.11 but I'm not 
  sure because it's a little bit late for arm-soc tree.
 
  Thanks for your ack, just note I will send to arm-soc tonight in my time.
 
 
  Can we get this series merge now in any of trees.
  Please let me if its required to rebase it.
 
  Regards,
  Yadwinder
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-07-22 Thread Vikas Sajjan
Hi Kukjin / Mike,

Can you apply this series

On Tue, Jul 16, 2013 at 7:10 PM, Yadwinder Singh Brar
yadi.bra...@gmail.com wrote:
 Hi Kukjin / Mike,

 On Tue, Jun 25, 2013 at 7:40 AM, Kukjin Kim kgene@samsung.com wrote:
 Mike Turquette wrote:

 Quoting Kukjin Kim (2013-06-24 08:02:39)
  On 06/22/13 01:31, Mike Turquette wrote:
  
   Acked-by: Mike Turquettemturque...@linaro.org
  
   Or I can take this through the clk tree. Either way is fine.
  
  Mike, please go ahead taking this series into the clk tree for 3.11.

 Actually it might be better for you to take it. Patch #5 relies on
 drivers/clk/samsung/clk-exynos5420.c which only exists in the samsung
 tree.  It was introduced by commit 1609027f, clk: exynos5420: register
 clocks using common clock framework.

 Yes, you're right. It depends on soc-exynos5420 stuff in samsung tree.

 And these patches need to be applied with some fuzz to the clk tree due
 to other changes which are not in my tree and only present in yours.

 The best two options are either for you to take this series or it can be
 rebased on top of clk-next, after which I'll apply it.

 Let me try to send this series via samsung tree for v3.11 but I'm not sure 
 because it's a little bit late for arm-soc tree.

 Thanks for your ack, just note I will send to arm-soc tonight in my time.


 Can we get this series merge now in any of trees.
 Please let me if its required to rebase it.

 Regards,
 Yadwinder
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-07-22 Thread Mike Turquette
Quoting Vikas Sajjan (2013-07-21 23:15:27)
 Hi Kukjin / Mike,
 
 Can you apply this series

Let me apply this to see if the dependencies are taken care of now. If
so I'll just take it through the clk tree.

Regards,
Mike

 
 On Tue, Jul 16, 2013 at 7:10 PM, Yadwinder Singh Brar
 yadi.bra...@gmail.com wrote:
  Hi Kukjin / Mike,
 
  On Tue, Jun 25, 2013 at 7:40 AM, Kukjin Kim kgene@samsung.com wrote:
  Mike Turquette wrote:
 
  Quoting Kukjin Kim (2013-06-24 08:02:39)
   On 06/22/13 01:31, Mike Turquette wrote:
   
Acked-by: Mike Turquettemturque...@linaro.org
   
Or I can take this through the clk tree. Either way is fine.
   
   Mike, please go ahead taking this series into the clk tree for 3.11.
 
  Actually it might be better for you to take it. Patch #5 relies on
  drivers/clk/samsung/clk-exynos5420.c which only exists in the samsung
  tree.  It was introduced by commit 1609027f, clk: exynos5420: register
  clocks using common clock framework.
 
  Yes, you're right. It depends on soc-exynos5420 stuff in samsung tree.
 
  And these patches need to be applied with some fuzz to the clk tree due
  to other changes which are not in my tree and only present in yours.
 
  The best two options are either for you to take this series or it can be
  rebased on top of clk-next, after which I'll apply it.
 
  Let me try to send this series via samsung tree for v3.11 but I'm not sure 
  because it's a little bit late for arm-soc tree.
 
  Thanks for your ack, just note I will send to arm-soc tonight in my time.
 
 
  Can we get this series merge now in any of trees.
  Please let me if its required to rebase it.
 
  Regards,
  Yadwinder
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-24 Thread Kukjin Kim

On 06/22/13 01:31, Mike Turquette wrote:


Acked-by: Mike Turquettemturque...@linaro.org

Or I can take this through the clk tree. Either way is fine.


Mike, please go ahead taking this series into the clk tree for 3.11.

Thanks,
- Kukjin
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-24 Thread Kukjin Kim
Mike Turquette wrote:
 
 Quoting Kukjin Kim (2013-06-24 08:02:39)
  On 06/22/13 01:31, Mike Turquette wrote:
  
   Acked-by: Mike Turquettemturque...@linaro.org
  
   Or I can take this through the clk tree. Either way is fine.
  
  Mike, please go ahead taking this series into the clk tree for 3.11.
 
 Actually it might be better for you to take it. Patch #5 relies on
 drivers/clk/samsung/clk-exynos5420.c which only exists in the samsung
 tree.  It was introduced by commit 1609027f, clk: exynos5420: register
 clocks using common clock framework.
 
Yes, you're right. It depends on soc-exynos5420 stuff in samsung tree.

 And these patches need to be applied with some fuzz to the clk tree due
 to other changes which are not in my tree and only present in yours.
 
 The best two options are either for you to take this series or it can be
 rebased on top of clk-next, after which I'll apply it.
 
Let me try to send this series via samsung tree for v3.11 but I'm not sure 
because it's a little bit late for arm-soc tree.

Thanks for your ack, just note I will send to arm-soc tonight in my time.

- Kukjin

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-21 Thread Tomasz Figa
On Friday 21 of June 2013 10:18:01 Vikas Sajjan wrote:
 Hi Mr. Kim,
 
 On Thu, Jun 20, 2013 at 3:37 PM, Yadwinder Singh Brar
 
 yadi.bra...@gmail.com wrote:
  Hi Tomasz,
  
  On Thu, Jun 20, 2013 at 3:30 PM, Tomasz Figa t.f...@samsung.com 
wrote:
  Hi Yadwinder, Vikas,
  
  On Tuesday 11 of June 2013 15:01:05 Yadwinder Singh Brar wrote:
  This patch series does the following:
   1) Unifies the clk strutures  and registration function used for
   PLL35xx 
   PLL36xx, to factor out possible common code.
  
   2) Defines a common rate_table which will contain recommended p, m,
   s
  
  and k values for supported rates that needs to be changed for
  changing
  corresponding PLL's rate
  
   3) Adds set_rate() and round_rate() clk_ops for PLL35xx and PLL36xx
  
  changes since v6:
- Splited the patch adding samsung_clk_register_pll() into
definition
addition, SoC specific migration and cleanup patches.
- Addressed some NIT comments.
  
  This version looks good to me. Thanks for your work on improving this
  series again!
  
  Reviewed-by: Tomasz Figa t.f...@samsung.com
 
 Can you apply this series..

I think we need at least Mike's (clk subsystem maintainer) ACK here.

Mike, do you have any objections?

Best regards,
Tomasz

  Thanks for helping to improve this series.
  
  Thanks,
  Yadwinder
  
  Best regards,
  Tomasz
  
  changes since v5:
- Corrected to use rate table as specified in UM for
exynos5250 epll.
- Took care of exynos5420 as well, as new exynos5420 clk
driver came in
while rebasing on latest Kgene's for-next.

Since we spilted 1st patch into 2 different patches,
can we expect reviewed-by again for the same?
  
  changes since v4:
- Defined common samsung samsung_clk_register_pll() to
register a list
of PLL and used a struct samsung_pll_clock for passing
intialisation
data instead of passing as arguments. Now passing LOCK as well
as CON0
offset as intialisation data.
- Calculated length of rate table while registering PLL
instead of
getting it as intialisation data.
  
  changes since v3:
- Used __clk_lookup() instead of adding alias for mout_vpllsrc
- Added check for changing only M value in
samsung_pll36xx_set_rate()
- Modified samsung_pll35xx_mp_change() 
samsung_pll35xx_set_rate()
to improve readabilty.
- Made the input rate_table as __init_data which is to be
provided
  
  while
  
  registering PLL and made a copy of that table while registering, so
  that if multiple tables are their, they can be freed after getting
  the P, M, S, K setting values from required one.
  
  changes since v2:
- Added new patch to reorder the MUX registration for
mout_vpllsrc MUX
before the PLL registrations. And to add the alias for the
mout_vpllsrc
  
  MUX. - Added a check to confirm parent rate while registrating the
  PLL
  rate tables.
  
  changes since v1:
- removed sorting and bsearch
- modified the definition of struct samsung_pll_rate_table
- added generic round_rate()
- rectified the ops assignment for rate table passed as NULL
  
  Is rebased on branch kgene's for-next
  https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git
  /log/ ?h=for-next
  
  Vikas Sajjan (3):
clk: samsung: Add set_rate() clk_ops for PLL36xx
clk: samsung: Reorder MUX registration for mout_vpllsrc
clk: samsung: Add EPLL and VPLL freq table for exynos5250 SoC
  
  Yadwinder Singh Brar (8):
clk: samsung: Introduce a common samsung_clk_pll struct
clk: samsung: Define a common samsung_clk_register_pll()
clk: samsung: Migrate exynos5250 to use common

  samsung_clk_register_pll()

clk: samsung: Migrate exynos4 to use common

  samsung_clk_register_pll()

clk: samsung: Migrate exynos5420 to use common

  samsung_clk_register_pll()

clk: samsung: Remove unused pll registeration code for pll35xx and

  pll36xx

clk: samsung: Add support to register rate_table for samsung plls
clk: samsung: Add set_rate() clk_ops for PLL35xx
   
   drivers/clk/samsung/clk-exynos4.c|   40 +++--
   drivers/clk/samsung/clk-exynos5250.c |  101 +--
   drivers/clk/samsung/clk-exynos5420.c |   86 ++---
   drivers/clk/samsung/clk-pll.c|  339
  
  ++ drivers/clk/samsung/clk-pll.h
  
  |   38 -
   
   drivers/clk/samsung/clk.h|   53 ++
   6 files changed, 519 insertions(+), 138 deletions(-)
  
  --
  To unsubscribe from this list: send the line unsubscribe
  linux-samsung-soc in the body of a message to
  majord...@vger.kernel.org
  More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
  --
  To unsubscribe from this list: send the line unsubscribe
  

[PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-20 Thread Yadwinder Singh Brar
This patch series does the following:

 1) Unifies the clk strutures  and registration function used for PLL35xx 
PLL36xx, to factor out possible common code.

 2) Defines a common rate_table which will contain recommended p, m, s and k
values for supported rates that needs to be changed for changing
corresponding PLL's rate

 3) Adds set_rate() and round_rate() clk_ops for PLL35xx and PLL36xx

changes since v6:
- Splited the patch adding samsung_clk_register_pll() into definition
addition, SoC specific migration and cleanup patches.
- Addressed some NIT comments.

changes since v5:
- Corrected to use rate table as specified in UM for exynos5250 epll.
- Took care of exynos5420 as well, as new exynos5420 clk driver came in
while rebasing on latest Kgene's for-next.

Since we spilted 1st patch into 2 different patches,
can we expect reviewed-by again for the same? 

changes since v4:
- Defined common samsung samsung_clk_register_pll() to register a list
of PLL and used a struct samsung_pll_clock for passing intialisation
data instead of passing as arguments. Now passing LOCK as well as CON0
offset as intialisation data.
- Calculated length of rate table while registering PLL instead of
getting it as intialisation data.

changes since v3:
- Used __clk_lookup() instead of adding alias for mout_vpllsrc
- Added check for changing only M value in samsung_pll36xx_set_rate()
- Modified samsung_pll35xx_mp_change()  samsung_pll35xx_set_rate()
to improve readabilty.
- Made the input rate_table as __init_data which is to be provided while
registering PLL and made a copy of that table while registering, so
that if multiple tables are their, they can be freed after getting the
P, M, S, K setting values from required one.

changes since v2:
- Added new patch to reorder the MUX registration for mout_vpllsrc MUX
before the PLL registrations. And to add the alias for the mout_vpllsrc 
MUX.
- Added a check to confirm parent rate while registrating the PLL
rate tables.

changes since v1:
- removed sorting and bsearch
- modified the definition of struct samsung_pll_rate_table
- added generic round_rate()
- rectified the ops assignment for rate table passed as NULL

Is rebased on branch kgene's for-next
https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/?h=for-next

Vikas Sajjan (3):
  clk: samsung: Add set_rate() clk_ops for PLL36xx
  clk: samsung: Reorder MUX registration for mout_vpllsrc
  clk: samsung: Add EPLL and VPLL freq table for exynos5250 SoC

Yadwinder Singh Brar (8):
  clk: samsung: Introduce a common samsung_clk_pll struct
  clk: samsung: Define a common samsung_clk_register_pll()
  clk: samsung: Migrate exynos5250 to use common
samsung_clk_register_pll()
  clk: samsung: Migrate exynos4 to use common
samsung_clk_register_pll()
  clk: samsung: Migrate exynos5420 to use common
samsung_clk_register_pll()
  clk: samsung: Remove unused pll registeration code for pll35xx and
pll36xx
  clk: samsung: Add support to register rate_table for samsung plls
  clk: samsung: Add set_rate() clk_ops for PLL35xx

 drivers/clk/samsung/clk-exynos4.c|   40 +++--
 drivers/clk/samsung/clk-exynos5250.c |  101 +--
 drivers/clk/samsung/clk-exynos5420.c |   86 ++---
 drivers/clk/samsung/clk-pll.c|  339 ++
 drivers/clk/samsung/clk-pll.h|   38 -
 drivers/clk/samsung/clk.h|   53 ++
 6 files changed, 519 insertions(+), 138 deletions(-)

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-20 Thread Tomasz Figa
Hi Yadwinder, Vikas,

On Tuesday 11 of June 2013 15:01:05 Yadwinder Singh Brar wrote:
 This patch series does the following:
 
  1) Unifies the clk strutures  and registration function used for PLL35xx
  PLL36xx, to factor out possible common code.
 
  2) Defines a common rate_table which will contain recommended p, m, s
 and k values for supported rates that needs to be changed for changing
 corresponding PLL's rate
 
  3) Adds set_rate() and round_rate() clk_ops for PLL35xx and PLL36xx
 
 changes since v6:
   - Splited the patch adding samsung_clk_register_pll() into definition
   addition, SoC specific migration and cleanup patches.
   - Addressed some NIT comments.

This version looks good to me. Thanks for your work on improving this 
series again!

Reviewed-by: Tomasz Figa t.f...@samsung.com

Best regards,
Tomasz

 changes since v5:
   - Corrected to use rate table as specified in UM for exynos5250 epll.
   - Took care of exynos5420 as well, as new exynos5420 clk driver came in
   while rebasing on latest Kgene's for-next.
 
   Since we spilted 1st patch into 2 different patches,
   can we expect reviewed-by again for the same?
 
 changes since v4:
   - Defined common samsung samsung_clk_register_pll() to register a list
   of PLL and used a struct samsung_pll_clock for passing intialisation
   data instead of passing as arguments. Now passing LOCK as well as CON0
   offset as intialisation data.
   - Calculated length of rate table while registering PLL instead of
   getting it as intialisation data.
 
 changes since v3:
   - Used __clk_lookup() instead of adding alias for mout_vpllsrc
   - Added check for changing only M value in samsung_pll36xx_set_rate()
   - Modified samsung_pll35xx_mp_change()  samsung_pll35xx_set_rate()
   to improve readabilty.
   - Made the input rate_table as __init_data which is to be provided 
while
 registering PLL and made a copy of that table while registering, so that
 if multiple tables are their, they can be freed after getting the P, M,
 S, K setting values from required one.
 
 changes since v2:
   - Added new patch to reorder the MUX registration for mout_vpllsrc MUX
   before the PLL registrations. And to add the alias for the mout_vpllsrc
 MUX. - Added a check to confirm parent rate while registrating the PLL
 rate tables.
 
 changes since v1:
   - removed sorting and bsearch
   - modified the definition of struct samsung_pll_rate_table
   - added generic round_rate()
   - rectified the ops assignment for rate table passed as NULL
 
 Is rebased on branch kgene's for-next
 https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/
 ?h=for-next
 
 Vikas Sajjan (3):
   clk: samsung: Add set_rate() clk_ops for PLL36xx
   clk: samsung: Reorder MUX registration for mout_vpllsrc
   clk: samsung: Add EPLL and VPLL freq table for exynos5250 SoC
 
 Yadwinder Singh Brar (8):
   clk: samsung: Introduce a common samsung_clk_pll struct
   clk: samsung: Define a common samsung_clk_register_pll()
   clk: samsung: Migrate exynos5250 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos4 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos5420 to use common
 samsung_clk_register_pll()
   clk: samsung: Remove unused pll registeration code for pll35xx and
 pll36xx
   clk: samsung: Add support to register rate_table for samsung plls
   clk: samsung: Add set_rate() clk_ops for PLL35xx
 
  drivers/clk/samsung/clk-exynos4.c|   40 +++--
  drivers/clk/samsung/clk-exynos5250.c |  101 +--
  drivers/clk/samsung/clk-exynos5420.c |   86 ++---
  drivers/clk/samsung/clk-pll.c|  339
 ++ drivers/clk/samsung/clk-pll.h   
 |   38 -
  drivers/clk/samsung/clk.h|   53 ++
  6 files changed, 519 insertions(+), 138 deletions(-)
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-20 Thread Yadwinder Singh Brar
Hi Tomasz,

On Thu, Jun 20, 2013 at 3:30 PM, Tomasz Figa t.f...@samsung.com wrote:
 Hi Yadwinder, Vikas,

 On Tuesday 11 of June 2013 15:01:05 Yadwinder Singh Brar wrote:
 This patch series does the following:

  1) Unifies the clk strutures  and registration function used for PLL35xx
  PLL36xx, to factor out possible common code.

  2) Defines a common rate_table which will contain recommended p, m, s
 and k values for supported rates that needs to be changed for changing
 corresponding PLL's rate

  3) Adds set_rate() and round_rate() clk_ops for PLL35xx and PLL36xx

 changes since v6:
   - Splited the patch adding samsung_clk_register_pll() into definition
   addition, SoC specific migration and cleanup patches.
   - Addressed some NIT comments.

 This version looks good to me. Thanks for your work on improving this
 series again!

 Reviewed-by: Tomasz Figa t.f...@samsung.com


Thanks for helping to improve this series.

Thanks,
Yadwinder

 Best regards,
 Tomasz

 changes since v5:
   - Corrected to use rate table as specified in UM for exynos5250 epll.
   - Took care of exynos5420 as well, as new exynos5420 clk driver came in
   while rebasing on latest Kgene's for-next.

   Since we spilted 1st patch into 2 different patches,
   can we expect reviewed-by again for the same?

 changes since v4:
   - Defined common samsung samsung_clk_register_pll() to register a list
   of PLL and used a struct samsung_pll_clock for passing intialisation
   data instead of passing as arguments. Now passing LOCK as well as CON0
   offset as intialisation data.
   - Calculated length of rate table while registering PLL instead of
   getting it as intialisation data.

 changes since v3:
   - Used __clk_lookup() instead of adding alias for mout_vpllsrc
   - Added check for changing only M value in samsung_pll36xx_set_rate()
   - Modified samsung_pll35xx_mp_change()  samsung_pll35xx_set_rate()
   to improve readabilty.
   - Made the input rate_table as __init_data which is to be provided
 while
 registering PLL and made a copy of that table while registering, so that
 if multiple tables are their, they can be freed after getting the P, M,
 S, K setting values from required one.

 changes since v2:
   - Added new patch to reorder the MUX registration for mout_vpllsrc MUX
   before the PLL registrations. And to add the alias for the mout_vpllsrc
 MUX. - Added a check to confirm parent rate while registrating the PLL
 rate tables.

 changes since v1:
   - removed sorting and bsearch
   - modified the definition of struct samsung_pll_rate_table
   - added generic round_rate()
   - rectified the ops assignment for rate table passed as NULL

 Is rebased on branch kgene's for-next
 https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/
 ?h=for-next

 Vikas Sajjan (3):
   clk: samsung: Add set_rate() clk_ops for PLL36xx
   clk: samsung: Reorder MUX registration for mout_vpllsrc
   clk: samsung: Add EPLL and VPLL freq table for exynos5250 SoC

 Yadwinder Singh Brar (8):
   clk: samsung: Introduce a common samsung_clk_pll struct
   clk: samsung: Define a common samsung_clk_register_pll()
   clk: samsung: Migrate exynos5250 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos4 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos5420 to use common
 samsung_clk_register_pll()
   clk: samsung: Remove unused pll registeration code for pll35xx and
 pll36xx
   clk: samsung: Add support to register rate_table for samsung plls
   clk: samsung: Add set_rate() clk_ops for PLL35xx

  drivers/clk/samsung/clk-exynos4.c|   40 +++--
  drivers/clk/samsung/clk-exynos5250.c |  101 +--
  drivers/clk/samsung/clk-exynos5420.c |   86 ++---
  drivers/clk/samsung/clk-pll.c|  339
 ++ drivers/clk/samsung/clk-pll.h
 |   38 -
  drivers/clk/samsung/clk.h|   53 ++
  6 files changed, 519 insertions(+), 138 deletions(-)
 --
 To unsubscribe from this list: send the line unsubscribe linux-samsung-soc 
 in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 00/11] Add generic set_rate clk_ops for PLL35xx and PLL36xx for samsung SoCs

2013-06-20 Thread Vikas Sajjan
Hi Mr. Kim,

On Thu, Jun 20, 2013 at 3:37 PM, Yadwinder Singh Brar
yadi.bra...@gmail.com wrote:
 Hi Tomasz,

 On Thu, Jun 20, 2013 at 3:30 PM, Tomasz Figa t.f...@samsung.com wrote:
 Hi Yadwinder, Vikas,

 On Tuesday 11 of June 2013 15:01:05 Yadwinder Singh Brar wrote:
 This patch series does the following:

  1) Unifies the clk strutures  and registration function used for PLL35xx
  PLL36xx, to factor out possible common code.

  2) Defines a common rate_table which will contain recommended p, m, s
 and k values for supported rates that needs to be changed for changing
 corresponding PLL's rate

  3) Adds set_rate() and round_rate() clk_ops for PLL35xx and PLL36xx

 changes since v6:
   - Splited the patch adding samsung_clk_register_pll() into definition
   addition, SoC specific migration and cleanup patches.
   - Addressed some NIT comments.

 This version looks good to me. Thanks for your work on improving this
 series again!

 Reviewed-by: Tomasz Figa t.f...@samsung.com



Can you apply this series..



 Thanks for helping to improve this series.

 Thanks,
 Yadwinder

 Best regards,
 Tomasz

 changes since v5:
   - Corrected to use rate table as specified in UM for exynos5250 epll.
   - Took care of exynos5420 as well, as new exynos5420 clk driver came 
 in
   while rebasing on latest Kgene's for-next.

   Since we spilted 1st patch into 2 different patches,
   can we expect reviewed-by again for the same?

 changes since v4:
   - Defined common samsung samsung_clk_register_pll() to register a list
   of PLL and used a struct samsung_pll_clock for passing intialisation
   data instead of passing as arguments. Now passing LOCK as well as CON0
   offset as intialisation data.
   - Calculated length of rate table while registering PLL instead of
   getting it as intialisation data.

 changes since v3:
   - Used __clk_lookup() instead of adding alias for mout_vpllsrc
   - Added check for changing only M value in samsung_pll36xx_set_rate()
   - Modified samsung_pll35xx_mp_change()  samsung_pll35xx_set_rate()
   to improve readabilty.
   - Made the input rate_table as __init_data which is to be provided
 while
 registering PLL and made a copy of that table while registering, so that
 if multiple tables are their, they can be freed after getting the P, M,
 S, K setting values from required one.

 changes since v2:
   - Added new patch to reorder the MUX registration for mout_vpllsrc MUX
   before the PLL registrations. And to add the alias for the 
 mout_vpllsrc
 MUX. - Added a check to confirm parent rate while registrating the PLL
 rate tables.

 changes since v1:
   - removed sorting and bsearch
   - modified the definition of struct samsung_pll_rate_table
   - added generic round_rate()
   - rectified the ops assignment for rate table passed as NULL

 Is rebased on branch kgene's for-next
 https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/
 ?h=for-next

 Vikas Sajjan (3):
   clk: samsung: Add set_rate() clk_ops for PLL36xx
   clk: samsung: Reorder MUX registration for mout_vpllsrc
   clk: samsung: Add EPLL and VPLL freq table for exynos5250 SoC

 Yadwinder Singh Brar (8):
   clk: samsung: Introduce a common samsung_clk_pll struct
   clk: samsung: Define a common samsung_clk_register_pll()
   clk: samsung: Migrate exynos5250 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos4 to use common
 samsung_clk_register_pll()
   clk: samsung: Migrate exynos5420 to use common
 samsung_clk_register_pll()
   clk: samsung: Remove unused pll registeration code for pll35xx and
 pll36xx
   clk: samsung: Add support to register rate_table for samsung plls
   clk: samsung: Add set_rate() clk_ops for PLL35xx

  drivers/clk/samsung/clk-exynos4.c|   40 +++--
  drivers/clk/samsung/clk-exynos5250.c |  101 +--
  drivers/clk/samsung/clk-exynos5420.c |   86 ++---
  drivers/clk/samsung/clk-pll.c|  339
 ++ drivers/clk/samsung/clk-pll.h
 |   38 -
  drivers/clk/samsung/clk.h|   53 ++
  6 files changed, 519 insertions(+), 138 deletions(-)
 --
 To unsubscribe from this list: send the line unsubscribe linux-samsung-soc 
 in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 --
 To unsubscribe from this list: send the line unsubscribe linux-samsung-soc 
 in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html