Re: [RFC 0/4] Add basic support for ASV
Hi MyungJoo, Thanks for your review. On Wed, Nov 13, 2013 at 11:33 AM, MyungJoo Ham myungjoo@samsung.com wrote: On Mon, Nov 11, 2013 at 11:27 PM, Yadwinder Singh Brar yadi.bra...@gmail.com wrote: gentle ping for suggestions/reviews .. On Wed, Sep 11, 2013 at 8:14 PM, Yadwinder Singh Brar yadi.b...@samsung.com wrote: This series is to add basic common infrastructure for ASV. Basically ASV is a technique used on samsung SoCs, which provides the recommended supply voltage for dvfs of arm, mif etc. For a given operating frequency, the voltage is recommended based on SoC's ASV group. ASV group gets fussed on SoCs during process of mass production. ASV is an instance of AVS. Please recondier and try to reuse what's already there (drivers/power/avs) Quote from drivers/power/avs/Kconfig: At a given operating point the voltage is adapted depending on static factors (chip manufacturing process) and dynamic factors (temperature depending performance). It seems that the current ASV is subset of AVS. Yes, both seems to be for similar purpose but still significantly different from each other. I would like to describe/differentiate it little bit here for explanation for others, that it is AVS(adaptive voltage scaling) which seems to be an IP/module doing adaptive voltage scaling where as ours is ASV(Adaptive scaling voltage), it provides only (adaptive) voltage for scaling to other drivers. Although the current implementation of AVS does not provide significant infrastructure to its sisters, we may start by sharing the directory. Yes, we can just share directory only right now, other then that i didn't see any thing can be common. Firstly i had put in that directory itself, later being doubtful i moved it to new directory but we can share directory. Any ideas/comments from others ? Regards, Yadwinder Added Jean Pihet, who has submitted AVS (TI). Cheers, MyungJoo. This series includes: - basic common infrastructue for ASV. It provides common APIs for user drivers like cpufreq devfreq and and an interface for SoC specific drivers to register ASV members(instances) - a common platform driver to register ASV members for exynos SoCs - an example providing minimal support (only for ARM ASV) for exynos5250 chips Its just basic skelton which I wanted to get it reviewed or discussed in early stage, before going ahead on further development based on it. Presently example is based on static ASV table provided in SoC specific file, which I expects to go into DT. But exactly how and where needs to be discussed, may be in next revisions once we get through the basic skelton. Also the location of driver in kernel may also seem odd to someone and many more things :). Looking for your valuable reviews and suggestions. Thanks Yadwinder Singh Brar (4): power: asv: Add common ASV support for samsung SoCs power: asv: Add a common asv driver for exynos SoCs. power: asv: Add support for exynos5250 arm: exynos5: Register static platform device for ASV. arch/arm/mach-exynos/mach-exynos5-dt.c |3 + drivers/power/Kconfig|1 + drivers/power/Makefile |1 + drivers/power/asv/Kconfig| 24 drivers/power/asv/Makefile |2 + drivers/power/asv/exynos-asv.c | 81 ++ drivers/power/asv/exynos-asv.h | 22 drivers/power/asv/exynos5250-asv.c | 141 drivers/power/asv/samsung-asv.c | 175 ++ include/linux/power/samsung-asv-driver.h | 61 +++ include/linux/power/samsung-asv.h| 37 +++ 11 files changed, 548 insertions(+), 0 deletions(-) create mode 100644 drivers/power/asv/Kconfig create mode 100644 drivers/power/asv/Makefile create mode 100644 drivers/power/asv/exynos-asv.c create mode 100644 drivers/power/asv/exynos-asv.h create mode 100644 drivers/power/asv/exynos5250-asv.c create mode 100644 drivers/power/asv/samsung-asv.c create mode 100644 include/linux/power/samsung-asv-driver.h create mode 100644 include/linux/power/samsung-asv.h ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- MyungJoo Ham, Ph.D. System S/W Lab, S/W Center, Samsung Electronics -- 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: [RFC 0/4] Add basic support for ASV
Hi Yadwinder, On Monday 11 of November 2013 23:27:08 Yadwinder Singh Brar wrote: gentle ping for suggestions/reviews .. Hmm, I must have somehow missed this series when you orignally sent it. Can I ask you to resend it, as I don't have it in my e-mail client archive any longer? While at it, I would also ask you to add my private e-mail address (this one) and linux-pm mailing list to Cc list. Thanks in advance. Best regards, Tomasz On Wed, Sep 11, 2013 at 8:14 PM, Yadwinder Singh Brar yadi.b...@samsung.com wrote: This series is to add basic common infrastructure for ASV. Basically ASV is a technique used on samsung SoCs, which provides the recommended supply voltage for dvfs of arm, mif etc. For a given operating frequency, the voltage is recommended based on SoC's ASV group. ASV group gets fussed on SoCs during process of mass production. This series includes: - basic common infrastructue for ASV. It provides common APIs for user drivers like cpufreq devfreq and and an interface for SoC specific drivers to register ASV members(instances) - a common platform driver to register ASV members for exynos SoCs - an example providing minimal support (only for ARM ASV) for exynos5250 chips Its just basic skelton which I wanted to get it reviewed or discussed in early stage, before going ahead on further development based on it. Presently example is based on static ASV table provided in SoC specific file, which I expects to go into DT. But exactly how and where needs to be discussed, may be in next revisions once we get through the basic skelton. Also the location of driver in kernel may also seem odd to someone and many more things :). Looking for your valuable reviews and suggestions. Thanks Yadwinder Singh Brar (4): power: asv: Add common ASV support for samsung SoCs power: asv: Add a common asv driver for exynos SoCs. power: asv: Add support for exynos5250 arm: exynos5: Register static platform device for ASV. arch/arm/mach-exynos/mach-exynos5-dt.c |3 + drivers/power/Kconfig|1 + drivers/power/Makefile |1 + drivers/power/asv/Kconfig| 24 drivers/power/asv/Makefile |2 + drivers/power/asv/exynos-asv.c | 81 ++ drivers/power/asv/exynos-asv.h | 22 drivers/power/asv/exynos5250-asv.c | 141 drivers/power/asv/samsung-asv.c | 175 ++ include/linux/power/samsung-asv-driver.h | 61 +++ include/linux/power/samsung-asv.h| 37 +++ 11 files changed, 548 insertions(+), 0 deletions(-) create mode 100644 drivers/power/asv/Kconfig create mode 100644 drivers/power/asv/Makefile create mode 100644 drivers/power/asv/exynos-asv.c create mode 100644 drivers/power/asv/exynos-asv.h create mode 100644 drivers/power/asv/exynos5250-asv.c create mode 100644 drivers/power/asv/samsung-asv.c create mode 100644 include/linux/power/samsung-asv-driver.h create mode 100644 include/linux/power/samsung-asv.h ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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: [RFC 0/4] Add basic support for ASV
[adding Sachin] Hi Tomasz, On Fri, Nov 15, 2013 at 12:06 AM, Tomasz Figa tomasz.f...@gmail.com wrote: Hi Yadwinder, On Monday 11 of November 2013 23:27:08 Yadwinder Singh Brar wrote: gentle ping for suggestions/reviews .. Hmm, I must have somehow missed this series when you orignally sent it. Can I ask you to resend it, as I don't have it in my e-mail client archive any longer? While at it, I would also ask you to add my private e-mail address (this one) and linux-pm mailing list to Cc list. hmm, I am on travel still for few more days, so it may be difficult for me to resend it quickly. But I think Sachin may help me here. Sachin can you please rebase and repost 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: [RFC 0/4] Add basic support for ASV
On Friday 15 of November 2013 08:15:52 Yadwinder Singh Brar wrote: [adding Sachin] Hi Tomasz, On Fri, Nov 15, 2013 at 12:06 AM, Tomasz Figa tomasz.f...@gmail.com wrote: Hi Yadwinder, On Monday 11 of November 2013 23:27:08 Yadwinder Singh Brar wrote: gentle ping for suggestions/reviews .. Hmm, I must have somehow missed this series when you orignally sent it. Can I ask you to resend it, as I don't have it in my e-mail client archive any longer? While at it, I would also ask you to add my private e-mail address (this one) and linux-pm mailing list to Cc list. hmm, I am on travel still for few more days, so it may be difficult for me to resend it quickly. But I think Sachin may help me here. Sachin can you please rebase and repost it ? OK. No need to hurry. I will look at the patches on some ML archives and then only post my comments once you resend the series. Best regards, Tomasz -- 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: [RFC 0/4] Add basic support for ASV
On Mon, Nov 11, 2013 at 11:27 PM, Yadwinder Singh Brar yadi.bra...@gmail.com wrote: gentle ping for suggestions/reviews .. On Wed, Sep 11, 2013 at 8:14 PM, Yadwinder Singh Brar yadi.b...@samsung.com wrote: This series is to add basic common infrastructure for ASV. Basically ASV is a technique used on samsung SoCs, which provides the recommended supply voltage for dvfs of arm, mif etc. For a given operating frequency, the voltage is recommended based on SoC's ASV group. ASV group gets fussed on SoCs during process of mass production. ASV is an instance of AVS. Please recondier and try to reuse what's already there (drivers/power/avs) Quote from drivers/power/avs/Kconfig: At a given operating point the voltage is adapted depending on static factors (chip manufacturing process) and dynamic factors (temperature depending performance). It seems that the current ASV is subset of AVS. Although the current implementation of AVS does not provide significant infrastructure to its sisters, we may start by sharing the directory. Added Jean Pihet, who has submitted AVS (TI). Cheers, MyungJoo. This series includes: - basic common infrastructue for ASV. It provides common APIs for user drivers like cpufreq devfreq and and an interface for SoC specific drivers to register ASV members(instances) - a common platform driver to register ASV members for exynos SoCs - an example providing minimal support (only for ARM ASV) for exynos5250 chips Its just basic skelton which I wanted to get it reviewed or discussed in early stage, before going ahead on further development based on it. Presently example is based on static ASV table provided in SoC specific file, which I expects to go into DT. But exactly how and where needs to be discussed, may be in next revisions once we get through the basic skelton. Also the location of driver in kernel may also seem odd to someone and many more things :). Looking for your valuable reviews and suggestions. Thanks Yadwinder Singh Brar (4): power: asv: Add common ASV support for samsung SoCs power: asv: Add a common asv driver for exynos SoCs. power: asv: Add support for exynos5250 arm: exynos5: Register static platform device for ASV. arch/arm/mach-exynos/mach-exynos5-dt.c |3 + drivers/power/Kconfig|1 + drivers/power/Makefile |1 + drivers/power/asv/Kconfig| 24 drivers/power/asv/Makefile |2 + drivers/power/asv/exynos-asv.c | 81 ++ drivers/power/asv/exynos-asv.h | 22 drivers/power/asv/exynos5250-asv.c | 141 drivers/power/asv/samsung-asv.c | 175 ++ include/linux/power/samsung-asv-driver.h | 61 +++ include/linux/power/samsung-asv.h| 37 +++ 11 files changed, 548 insertions(+), 0 deletions(-) create mode 100644 drivers/power/asv/Kconfig create mode 100644 drivers/power/asv/Makefile create mode 100644 drivers/power/asv/exynos-asv.c create mode 100644 drivers/power/asv/exynos-asv.h create mode 100644 drivers/power/asv/exynos5250-asv.c create mode 100644 drivers/power/asv/samsung-asv.c create mode 100644 include/linux/power/samsung-asv-driver.h create mode 100644 include/linux/power/samsung-asv.h ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- MyungJoo Ham, Ph.D. System S/W Lab, S/W Center, Samsung Electronics -- 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: [RFC 0/4] Add basic support for ASV
gentle ping for suggestions/reviews .. On Wed, Sep 11, 2013 at 8:14 PM, Yadwinder Singh Brar yadi.b...@samsung.com wrote: This series is to add basic common infrastructure for ASV. Basically ASV is a technique used on samsung SoCs, which provides the recommended supply voltage for dvfs of arm, mif etc. For a given operating frequency, the voltage is recommended based on SoC's ASV group. ASV group gets fussed on SoCs during process of mass production. This series includes: - basic common infrastructue for ASV. It provides common APIs for user drivers like cpufreq devfreq and and an interface for SoC specific drivers to register ASV members(instances) - a common platform driver to register ASV members for exynos SoCs - an example providing minimal support (only for ARM ASV) for exynos5250 chips Its just basic skelton which I wanted to get it reviewed or discussed in early stage, before going ahead on further development based on it. Presently example is based on static ASV table provided in SoC specific file, which I expects to go into DT. But exactly how and where needs to be discussed, may be in next revisions once we get through the basic skelton. Also the location of driver in kernel may also seem odd to someone and many more things :). Looking for your valuable reviews and suggestions. Thanks Yadwinder Singh Brar (4): power: asv: Add common ASV support for samsung SoCs power: asv: Add a common asv driver for exynos SoCs. power: asv: Add support for exynos5250 arm: exynos5: Register static platform device for ASV. arch/arm/mach-exynos/mach-exynos5-dt.c |3 + drivers/power/Kconfig|1 + drivers/power/Makefile |1 + drivers/power/asv/Kconfig| 24 drivers/power/asv/Makefile |2 + drivers/power/asv/exynos-asv.c | 81 ++ drivers/power/asv/exynos-asv.h | 22 drivers/power/asv/exynos5250-asv.c | 141 drivers/power/asv/samsung-asv.c | 175 ++ include/linux/power/samsung-asv-driver.h | 61 +++ include/linux/power/samsung-asv.h| 37 +++ 11 files changed, 548 insertions(+), 0 deletions(-) create mode 100644 drivers/power/asv/Kconfig create mode 100644 drivers/power/asv/Makefile create mode 100644 drivers/power/asv/exynos-asv.c create mode 100644 drivers/power/asv/exynos-asv.h create mode 100644 drivers/power/asv/exynos5250-asv.c create mode 100644 drivers/power/asv/samsung-asv.c create mode 100644 include/linux/power/samsung-asv-driver.h create mode 100644 include/linux/power/samsung-asv.h ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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
[RFC 0/4] Add basic support for ASV
This series is to add basic common infrastructure for ASV. Basically ASV is a technique used on samsung SoCs, which provides the recommended supply voltage for dvfs of arm, mif etc. For a given operating frequency, the voltage is recommended based on SoC's ASV group. ASV group gets fussed on SoCs during process of mass production. This series includes: - basic common infrastructue for ASV. It provides common APIs for user drivers like cpufreq devfreq and and an interface for SoC specific drivers to register ASV members(instances) - a common platform driver to register ASV members for exynos SoCs - an example providing minimal support (only for ARM ASV) for exynos5250 chips Its just basic skelton which I wanted to get it reviewed or discussed in early stage, before going ahead on further development based on it. Presently example is based on static ASV table provided in SoC specific file, which I expects to go into DT. But exactly how and where needs to be discussed, may be in next revisions once we get through the basic skelton. Also the location of driver in kernel may also seem odd to someone and many more things :). Looking for your valuable reviews and suggestions. Thanks Yadwinder Singh Brar (4): power: asv: Add common ASV support for samsung SoCs power: asv: Add a common asv driver for exynos SoCs. power: asv: Add support for exynos5250 arm: exynos5: Register static platform device for ASV. arch/arm/mach-exynos/mach-exynos5-dt.c |3 + drivers/power/Kconfig|1 + drivers/power/Makefile |1 + drivers/power/asv/Kconfig| 24 drivers/power/asv/Makefile |2 + drivers/power/asv/exynos-asv.c | 81 ++ drivers/power/asv/exynos-asv.h | 22 drivers/power/asv/exynos5250-asv.c | 141 drivers/power/asv/samsung-asv.c | 175 ++ include/linux/power/samsung-asv-driver.h | 61 +++ include/linux/power/samsung-asv.h| 37 +++ 11 files changed, 548 insertions(+), 0 deletions(-) create mode 100644 drivers/power/asv/Kconfig create mode 100644 drivers/power/asv/Makefile create mode 100644 drivers/power/asv/exynos-asv.c create mode 100644 drivers/power/asv/exynos-asv.h create mode 100644 drivers/power/asv/exynos5250-asv.c create mode 100644 drivers/power/asv/samsung-asv.c create mode 100644 include/linux/power/samsung-asv-driver.h create mode 100644 include/linux/power/samsung-asv.h -- 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
