target/pscsi: Make pscsi_configure_device + target_release_session static
authorFengguang Wu <fengguang.wu@intel.com>
Tue, 9 Oct 2012 23:30:20 +0000 (07:30 +0800)
committerNicholas Bellinger <nab@linux-iscsi.org>
Wed, 7 Nov 2012 04:55:45 +0000 (20:55 -0800)
drivers/target/target_core_pscsi.c:464:5: sparse: symbol 'pscsi_configure_device'
was not declared. Should it be static?

FYI, there are new sparse warnings show up in

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git queue
head:   738b86ac5e56c645aa5b7bf49cb38e2a04c665f8
commit: 410aeee637c47bcf7e8dd7893347fe0811e07ab1 [47/51] target: kill struct se_subsystem_dev

vim +464 drivers/target/target_core_pscsi.c

410aeee6 Christoph Hellwig  2012-10-08 @464  int pscsi_configure_device(struct se_device *dev)
c66ac9db Nicholas Bellinger 2010-12-17  465  {
410aeee6 Christoph Hellwig  2012-10-08  466   struct se_hba *hba = dev->se_hba;
410aeee6 Christoph Hellwig  2012-10-08  467   struct pscsi_dev_virt *pdv = PSCSI_DEV(dev);
c66ac9db Nicholas Bellinger 2010-12-17  468   struct scsi_device *sd;
410aeee6 Christoph Hellwig  2012-10-08  469   struct pscsi_hba_virt *phv = dev->se_hba->hba_ptr;
c66ac9db Nicholas Bellinger 2010-12-17  470   struct Scsi_Host *sh = phv->phv_lld_host;
c66ac9db Nicholas Bellinger 2010-12-17  471   int legacy_mode_enable = 0;
410aeee6 Christoph Hellwig  2012-10-08  472   int ret;

Please consider folding the attached diff :-)

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_pscsi.c
drivers/target/target_core_transport.c

index dc10e29934d7ec65373b442bda34e232e451b9b3..c3bcc17575d020f7d6e3d054fc3c40562b486b5a 100644 (file)
@@ -461,7 +461,7 @@ static int pscsi_create_type_other(struct se_device *dev,
        return 0;
 }
 
-int pscsi_configure_device(struct se_device *dev)
+static int pscsi_configure_device(struct se_device *dev)
 {
        struct se_hba *hba = dev->se_hba;
        struct pscsi_dev_virt *pdv = PSCSI_DEV(dev);
index d6d24a47ed66e78b3c47a1a800355a550728b1f2..c80d88435f90871a1d5cac0dfccafbaf2106a739 100644 (file)
@@ -297,7 +297,7 @@ void transport_register_session(
 }
 EXPORT_SYMBOL(transport_register_session);
 
-void target_release_session(struct kref *kref)
+static void target_release_session(struct kref *kref)
 {
        struct se_session *se_sess = container_of(kref,
                        struct se_session, sess_kref);