Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752161AbbHMK6q (ORCPT ); Thu, 13 Aug 2015 06:58:46 -0400 Received: from mail-pd0-f176.google.com ([209.85.192.176]:36053 "EHLO mail-pd0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751692AbbHMK6o (ORCPT ); Thu, 13 Aug 2015 06:58:44 -0400 Date: Thu, 13 Aug 2015 16:28:39 +0530 From: Viresh Kumar To: Krzysztof Kozlowski Cc: Jonas Rabenstein , "Rafael J. Wysocki" , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Remove unselectable rule for arm-exynos-cpufreq.o Message-ID: <20150813105839.GT20238@linux> References: <1439461941-13953-1-git-send-email-jonas.rabenstein@studium.uni-erlangen.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1943 Lines: 44 On 13-08-15, 19:53, Krzysztof Kozlowski wrote: > 2015-08-13 19:32 GMT+09:00 Jonas Rabenstein > : > > With commit 2907ed4377d9 ("cpufreq: exynos: remove Exynos4x12 specific > > cpufreq driver support"), the last specific Exynos-cpufreq driver > > depending on ARM_EXYNOS_CPUFREQ was removed. As there is no > > Exynos-specific cpufreq driver depending on the code in > > drivers/cpufreq/exynos-cpufreq.c (selected by ARM_EXYNOS_CPUFREQ), > > this file was deleted and ARM_EXYNOS_CPUFREQ removed from Kconfig. > > After all, the rule in drivers/cpufreq/Makefile, to combine all > > specific cpufreq drivers depending on the generic code for Exynos into > > one objectfile is obsolete and can be removed. > > > > Remove unselectable rule for arm-exynos-cpufreq.o from > > drivers/cpufreq/Makefile, which - if selectable - would generate an > > always empty objectfile. > > That's a lot, really a lot of words, just to say that > ARM_EXYNOS_CPUFREQ option was removed and we should get rid of any > left-overs. The patch is of course valid... but could you trim this to > a short sentence giving the reason? Mentioned commit ID comes from > linux-next (or maintainer branch which may rebase) so it won't work > long-term. I think you can just leave the title of the commit. > > When re-spinning: > Reviewed-by: Krzysztof Kozlowski > > Viresh, > Do you want this to go through samsung-soc tree? That's Rafael's call, but even he can't take it right now. This patch has dependency on the samsung pull request. So it should go that way only. But if you are already late, then Rafael can apply this after 4.3-rc1 is out. -- viresh -- 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/