Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751709AbdH3RyU (ORCPT ); Wed, 30 Aug 2017 13:54:20 -0400 Received: from mail.skyhub.de ([5.9.137.197]:42018 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751410AbdH3RyS (ORCPT ); Wed, 30 Aug 2017 13:54:18 -0400 Date: Wed, 30 Aug 2017 19:54:06 +0200 From: Borislav Petkov To: Jan Glauber Cc: Mark Rutland , Will Deacon , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Suzuki K Poulose , David Daney Subject: Re: [RFC PATCH v9 1/7] edac: thunderx: Remove suspend/resume support Message-ID: <20170830175406.ygmathi3b5orzyqy@pd.tnic> References: <20170829131238.4988-1-jglauber@cavium.com> <20170829131238.4988-2-jglauber@cavium.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170829131238.4988-2-jglauber@cavium.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1298 Lines: 35 On Tue, Aug 29, 2017 at 03:12:32PM +0200, Jan Glauber wrote: > The memory controller on ThunderX/OcteonTX systems does not > support power management. Therefore remove the suspend/resume > callbacks. > > Signed-off-by: Jan Glauber > --- > drivers/edac/thunderx_edac.c | 21 --------------------- > 1 file changed, 21 deletions(-) Just when I thought I'd pick that one up now because it is removing stuff: drivers/edac/thunderx_edac.c:817:14: error: ‘thunderx_lmc_suspend’ undeclared here (not in a function) .suspend = thunderx_lmc_suspend, ^~~~~~~~~~~~~~~~~~~~ drivers/edac/thunderx_edac.c:818:14: error: ‘thunderx_lmc_resume’ undeclared here (not in a function) .resume = thunderx_lmc_resume, ^~~~~~~~~~~~~~~~~~~ scripts/Makefile.build:308: recipe for target 'drivers/edac/thunderx_edac.o' failed make[1]: *** [drivers/edac/thunderx_edac.o] Error 1 make[1]: *** Waiting for unfinished jobs.... Makefile:1682: recipe for target 'drivers/edac/' failed make: *** [drivers/edac/] Error 2 Please make sure you build and test every patch before submitting. You're lucky I can at least build arm64 on my x86 workstation. :-) Thx. -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.