From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Thumshirn Subject: Re: [PATCH 3/5] scsi: bnx2fc: clean up header definitions Date: Fri, 5 May 2017 12:33:51 +0200 Message-ID: References: <20170410171254.30367-1-bigeasy@linutronix.de> <20170410171254.30367-4-bigeasy@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:35365 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751752AbdEEKd6 (ORCPT ); Fri, 5 May 2017 06:33:58 -0400 Received: by mail-wm0-f67.google.com with SMTP id b10so477691wmh.2 for ; Fri, 05 May 2017 03:33:57 -0700 (PDT) In-Reply-To: <20170410171254.30367-4-bigeasy@linutronix.de> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Sebastian Andrzej Siewior , "Martin K . Petersen" , "James E.J. Bottomley" , linux-scsi@vger.kernel.org Cc: rt@linutronix.de, Lee Duncan , Chris Leech , Chad Dupuis , QLogic-Storage-Upstream@qlogic.com, Christoph Hellwig , Andrew Morton On 04/10/2017 07:12 PM, Sebastian Andrzej Siewior wrote: > - All symbols which are only used within one .c file are marked static > and removed from the bnx2fc.h file if possible. > > - the declarion of bnx2fc_percpu is moved into the header file > > This patch was only compile-tested due to -ENODEV. > > Cc: QLogic-Storage-Upstream@qlogic.com > Cc: Christoph Hellwig > Signed-off-by: Sebastian Andrzej Siewior > --- Reviewed-by: Johannes Thumshirn