From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sachin Kamat Subject: Re: [RFC v2 0/4] Add basic support for ASV Date: Mon, 18 Nov 2013 09:37:49 +0530 Message-ID: References: <1384515691-26299-1-git-send-email-sachin.kamat@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=EUC-KR Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org To: MyungJoo Ham Cc: "Rafael J. Wysocki" , Linux PM list , Kukjin Kim , Tomasz Figa , linux-arm-kernel , Yadwinder Singh Brar , linux-samsung-soc , LKML List-Id: linux-pm@vger.kernel.org Hi MyungJoo, On 18 November 2013 08:07, MyungJoo Ham wrot= e: > > 2013. 11. 15. =BF=C0=C8=C4 8:44=BF=A1 "Sachin Kamat" =B4=D4=C0=CC =C0=DB=BC=BA: > > >> >> Original cover letter from Yadwinder: >> This series is to add basic common infrastructure for ASV. >> Basically ASV is a technique used on samsung SoCs, which provides t= he >> recommended supply voltage for dvfs of arm, mif etc. For a given ope= rating >> 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 u= ser >> 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 exyno= s5250 >> chips >> >> Its just basic skelton which I wanted to get it reviewed or discusse= d in >> early stage, before going ahead on further development based on it. >> Presently example is based on static ASV table provided in SoC spec= ific >> 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. > > Hi, > > As I have commented on the previous thread of Yadwinder, please share= the > directory woth AVS, which is conceptually a superset of ASV. > > Ultimately, it would be best if you can supply an infra that can be s= hared > with the current AVS, which does not have any currently. > Yadwinder has already replied to your suggestion about his concerns. Pl= ease share your comments. --=20 With warm regards, Sachin