From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1756692AbcHCHxw (ORCPT <rfc822;w@1wt.eu>);
	Wed, 3 Aug 2016 03:53:52 -0400
Received: from mout.kundenserver.de ([212.227.126.134]:65356 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1754377AbcHCHxI (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 3 Aug 2016 03:53:08 -0400
From: Arnd Bergmann <arnd@arndb.de>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, Anil Gurumurthy <anil.gurumurthy@qlogic.com>,
        Sudarsana Kalluru <sudarsana.kalluru@qlogic.com>,
        "James E . J . Bottomley" <jejb@linux.vnet.ibm.com>,
        "Martin K . Petersen" <martin.petersen@oracle.com>,
        linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
        Baoyou Xie <baoyou.xie@linaro.org>
Subject: Re: [PATCH 1/5] bfa: mark symbols static where possible
Date: Wed, 03 Aug 2016 09:46:01 +0200
Message-ID: <4472099.0XASieJnNU@wuerfel>
User-Agent: KMail/5.1.3 (Linux/4.4.0-31-generic; KDE/5.18.0; x86_64; ; )
In-Reply-To: <201608031208.Y8UqZJYW%fengguang.wu@intel.com>
References: <201608031208.Y8UqZJYW%fengguang.wu@intel.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:mfm4m5+z60iiw6gWJOF0O+8MNRLKYuTmGGO7/qgmL3lHUr+yzF/
 1/vSeAfGiQ5rO+JnPfmYL+w8wG9Q9GBijf1Kn0pYNPzD0O86gGTdDPuSv/jPIJ8px6DBlVU
 xZWnAWLmIg36g5puheHc/R612WjvUC3Vnxb2ssx8IPd+pPMnsYqYOcMIPqqB0KnfEnLiTc8
 vz8JpbJvVmRUcu3N7G0gA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:8tNi2p8XboY=:P5/vMttYBOZcdXQDa+IeJf
 0khQJTkkv7OPMUh8OVGqZBSGOhXwmmkl0yrEr6WppIGsZK2v8DwuwqfXLJQZnClWPQSRmAeTv
 GGUo5DRzBD0b82zYgO+DXG7inJMxEk4R1EVJY89AaPtJdu5M+X1v+c8PQmW65i/WIEdjFG0ge
 GXnLrE9jESLomVgl+W8mSRz6yn+6ZQ0nZyG4lCzuf/xKYBDFaiBNx8qYmCkh2xrO0lI66wjVs
 GbZlKuJoqmpDmsZ63VSGGujni1nHhzjeQUTctnwtKUAxqXHKaMvqYpiOsEn95WFSMV9lSUwQy
 wpY0YzyXNrYC3bhxtL90X6mJbK3x1r05/TTUEuHatEcvg5hDjNGLlRGZzemcPGEWJuM+ZwXxG
 jA9x9Ribd9MS+z7np1WsOry8Zbt8dHQlhb8Rt/J0adE+Kg2tbu9eI0yIbXZVnpLZmULLWE5zj
 we/zPzf5ptVwMwUjRgKW7NgR8897evSq92xUg2TPFM/dHqbu3qMGKFm9ZKTsiG1KsrnIA7qi0
 Im/KGl3X3U9IuRvkvmCFqSlzOD/fNr32UzMa16KIYI1E3Y9xdBd8FoAUY+UAEtzKWUJKmnnBr
 TOUIHk+G+3NwDn6EvVxPpAUqyk/dkC3zfEF2nbdm8NCcmQLYKBVJrIdYB2ZAVFr87+7Y7g/Ot
 bH1mDsebSzmXN4RJCzP94nMs4b/DE82qEhcKAE4w0MZt+ZJ+16mDY5MTaexrkSGyD10GXCtsL
 4S8rgXtO3rm7OkR7
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wednesday, August 3, 2016 12:45:50 PM CEST kbuild test robot wrote:
> All errors (new ones prefixed by >>):
> 
>    drivers/scsi/bfa/bfad.c: In function 'bfad_drv_uninit':
> >> drivers/scsi/bfa/bfad.c:888:2: error: implicit declaration of function 'bfa_isr_disable' [-Werror=implicit-function-declaration]
>      bfa_isr_disable(&bfad->bfa);
>      ^
>    cc1: some warnings being treated as errors
> 

Ah right: This is called in a function that gets removed in patch 5,
after which it is actually local. I made a mistake here when I went
through a repeated search for local functions.

I'll wait for other comments first before I send a fixed version.

	Arnd