On !x86 we currently don't read the existing support flags:

/home/benh/linux-3.14.13/drivers/scsi/hpsa.c:4375:17: warning: 'driver_support' 
is used uninitialized in this function [-Wuninitialized]
  driver_support |= ENABLE_UNIT_ATTN;
                 ^

Signed-off-by: Ben Hutchings <b...@decadent.org.uk>
Fixes: 28e134464734 ("[SCSI] hpsa: enable unit attention reporting")
Cc: <sta...@vger.kernel.org> # 3.14+
---
Compile-tested only.

Ben.

 drivers/scsi/hpsa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 9a6e4a2..7c3ff51 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -6350,9 +6350,9 @@ static inline void hpsa_set_driver_support_bits(struct 
ctlr_info *h)
 {
        u32 driver_support;
 
+       driver_support = readl(&(h->cfgtable->driver_support));
 #ifdef CONFIG_X86
        /* Need to enable prefetch in the SCSI core for 6400 in x86 */
-       driver_support = readl(&(h->cfgtable->driver_support));
        driver_support |= ENABLE_SCSI_PREFETCH;
 #endif
        driver_support |= ENABLE_UNIT_ATTN;

-- 
Ben Hutchings
Make three consecutive correct guesses and you will be considered an expert.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to