From patchwork Tue Aug 26 05:25:05 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chander Kashyap X-Patchwork-Id: 4778211 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 2DEA99F2E8 for ; Tue, 26 Aug 2014 05:27:49 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 4C39720142 for ; Tue, 26 Aug 2014 05:27:48 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 704D020131 for ; Tue, 26 Aug 2014 05:27:47 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1XM9Ge-00031z-L2; Tue, 26 Aug 2014 05:26:04 +0000 Received: from mail-ob0-x230.google.com ([2607:f8b0:4003:c01::230]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XM9G3-00007y-Iy for linux-arm-kernel@lists.infradead.org; Tue, 26 Aug 2014 05:25:28 +0000 Received: by mail-ob0-f176.google.com with SMTP id wo20so11289404obc.21 for ; Mon, 25 Aug 2014 22:25:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=IpcAjEZ5HrC6zxWfy8jMUkmTyXSTvvqBDx7+UYizNE4=; b=xZGTsFjP1wDj37LvJV3sSlb3ZLASG9bOec2PPKolqOYppwoO0MXKc2IUh5ZKwb/Qsw uTSJkhJAxZDRFfFeRqqBevHxeZ0oolKqCvtZGPbdsjjVvG6Bl3gInmihDCuZaZ8LlaYk KnXfGsK+HE7tVYLRBRjNjA5w50IJfQxdu17yIKHDUCTk727loq0jHDX99H/LEq6V0zTj PL+81Pl/liFF5NwdEJcNkEwfsLGop3+QBjY3OJaHP7VMzm9oDQZreq6aOwAidzmGWOgn JYi1hI41+3NOZqomEfXDpS3sTnV7i/NQb5kYBH2Kwxn4IfmiocAXSQGDqEezhYvmeNIe 9PiA== MIME-Version: 1.0 X-Received: by 10.60.74.232 with SMTP id x8mr59746oev.84.1409030705781; Mon, 25 Aug 2014 22:25:05 -0700 (PDT) Received: by 10.183.8.6 with HTTP; Mon, 25 Aug 2014 22:25:05 -0700 (PDT) In-Reply-To: <7h8umc60we.fsf@paris.lan> References: <1406707663-16656-1-git-send-email-thomas.ab@samsung.com> <1406707663-16656-5-git-send-email-thomas.ab@samsung.com> <53DA8BB9.6020702@samsung.com> <53DA8D85.3050106@gmail.com> <7hbnrc6px3.fsf@paris.lan> <53F7DA13.6090305@gmail.com> <7h8umc60we.fsf@paris.lan> Date: Tue, 26 Aug 2014 10:55:05 +0530 X-Google-Sender-Auth: ZtvquZaCzjmueaCHxHqjgSBhFbs Message-ID: Subject: Re: [PATCH v9 4/6] ARM: Exynos: switch to using generic cpufreq driver for Exynos4210/5250/5420 From: Chander Kashyap To: Kevin Hilman X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20140825_222527_691351_E7DB37B2 X-CRM114-Status: GOOD ( 18.84 ) X-Spam-Score: 0.0 (/) Cc: Chander Kashyap , Lukasz Majewski , Kukjin Kim , Mike Turquette , =?UTF-8?Q?Heiko_St=C3=BCbner?= , "linux-pm@vger.kernel.org" , Viresh Kumar , Bartlomiej Zolnierkiewicz , Tomasz Figa , Amit Kucheria , Chanwoo Choi , Thomas Abraham , "linux-samsung-soc@vger.kernel.org" , linaro-kernel@lists.linaro.org, "linux-arm-kernel@lists.infradead.org" , Marek Szyprowski X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=no version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi Kevin, On Mon, Aug 25, 2014 at 9:02 PM, Kevin Hilman wrote: > Hi Chander, > > Chander Kashyap writes: > > [...] > >>>> I'm trying it on the 5800/Chromebook2 and it's not terribly stable. I'm >>>> testing along with CPUidle, so there may be some untested interactions >>>> there as it seems a bit more stable without CPUidle enabled. >>>> >>>> I'd love to hear from anyone else that's testing CPUidle and CPUfreq >>>> together big.LITTLE 5420/5800, with or without the switcher. >> >> I have tested this patch series on SMDK5420 with cpuidle (with and >> without b.L switcher enabled). >> >> As of now voltage scaling support is not there in generic big-little >> cpufreq driver (arm_big_little.c). >> Hence need to tie arm and kfc voltages to highest level for testing. > >> Without this change stability issues are there, but with this change >> everything is stable. > > Can you clarify how you're setting the voltages to ensure stability? below is the diff : wip/exynos/integ > > Tomasz, I didn't mean to suggest this isn't ready for mainline. For the > 5420/5800 it seems cpufreq support is a new feature, so this isn't a > regression against previous (mainline) behavior. Maybe the big.LITTLE > cpufreq support should've been separated out from the cleanup since it's > more of a new feature, but that's up to you. > > Kevin > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel regards, Chander --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts @@ -225,7 +225,7 @@ buck2_reg: BUCK2 { regulator-name = "vdd_arm"; - regulator-min-microvolt = <800000>; + regulator-min-microvolt = <1500000>; regulator-max-microvolt = <1500000>; regulator-always-on; regulator-boot-on; @@ -260,7 +260,7 @@ buck6_reg: BUCK6 { regulator-name = "vdd_kfc"; - regulator-min-microvolt = <800000>; + regulator-min-microvolt = <1500000>; regulator-max-microvolt = <1500000>; regulator-always-on; regulator-boot-on;