Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756010AbbEZQLE (ORCPT ); Tue, 26 May 2015 12:11:04 -0400 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:33338 "EHLO opensource.wolfsonmicro.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756003AbbEZQKj (ORCPT ); Tue, 26 May 2015 12:10:39 -0400 Date: Tue, 26 May 2015 17:10:29 +0100 From: Richard Fitzgerald To: Mark Brown Cc: Lee Jones , linux-kernel@vger.kernel.org, patches@opensource.wolfsonmicro.com Subject: Re: [PATCH v6 1/1] mfd: arizona: Export functions to control subsystem DVFS Message-ID: <20150526161021.GA6487@opensource.wolfsonmicro.com> References: <1432047462-31860-1-git-send-email-rf@opensource.wolfsonmicro.com> <1432047462-31860-2-git-send-email-rf@opensource.wolfsonmicro.com> <20150526135205.GL11677@x1> <20150526152942.GU21577@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150526152942.GU21577@sirena.org.uk> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 936 Lines: 20 On Tue, May 26, 2015 at 04:29:42PM +0100, Mark Brown wrote: > On Tue, May 26, 2015 at 02:52:05PM +0100, Lee Jones wrote: > > > Shouldn't this DVFS dump live in drivers/cpufreq? > > cpufreq is specifically for CPUs (with the governors and everything) and > has its own problems here. There is devfreq which was crated for this > sort of thing (though more on a SoC level) though, not sure if it's a > good fit or not. devfreq does look as though it's at the wrong sort of level. Also looks like it could introduce more complexity than is justified. An simpler alternative if we don't want this code in mfd is that potentially this DVFS handling could be moved into the SoC codec driver. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/