From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752374AbdLFSVl (ORCPT ); Wed, 6 Dec 2017 13:21:41 -0500 Received: from smtp1.it.da.ut.ee ([193.40.5.66]:42164 "EHLO smtp1.it.da.ut.ee" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751894AbdLFSVk (ORCPT ); Wed, 6 Dec 2017 13:21:40 -0500 Date: Wed, 6 Dec 2017 20:21:34 +0200 (EET) From: Meelis Roos To: "Rafael J. Wysocki" cc: Viresh Kumar , Rafael Wysocki , Linux PM , Vincent Guittot , "4 . 14+" , Linux Kernel Mailing List Subject: Re: [PATCH] cpufreq: longhaul: Set transition_delay_us to 20 ms In-Reply-To: Message-ID: References: User-Agent: Alpine 2.21 (LRH 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > >> > diff --git a/drivers/cpufreq/longhaul.c b/drivers/cpufreq/longhaul.c > >> > index c46a12df40dd..56eafcb07859 100644 > >> > --- a/drivers/cpufreq/longhaul.c > >> > +++ b/drivers/cpufreq/longhaul.c > >> > @@ -894,7 +894,7 @@ static int longhaul_cpu_init(struct cpufreq_policy *policy) > >> > if ((longhaul_version != TYPE_LONGHAUL_V1) && (scale_voltage != 0)) > >> > longhaul_setup_voltagescaling(); > >> > > >> > - policy->cpuinfo.transition_latency = 200000; /* nsec */ > >> > + policy->transition_delay_us = 20000; /* usec */ > >> > > >> > return cpufreq_table_validate_and_show(policy, longhaul_table); > >> > } > >> > >> This patch also works on my EPIA-M board - tested 10+ times. > > > > An on the last try just after sending the mail, it hung again in the > > same way as before - so maybe 20 is on the edge of being good. > > OK, so can you please try to modify the patch to set > transition_delay_us to 30000, say, and see if that's reliable? 30000 was not reliable. I created root cron job @reboot sleep 120; /sbin/reboot and by the evening it was dead again. Will try 50000 tomorrow. -- Meelis Roos (mroos@linux.ee