From edbf05742004b3bc7bf62394fe6443deae769fd7 Mon Sep 17 00:00:00 2001 From: Larry Finger Date: Sat, 19 Apr 2008 17:52:12 +0200 Subject: [PATCH] ssb: Fix all-ones boardflags In the SSB SPROM a field set to all ones means the value is not defined in the SPROM. In case of the boardflags, we need to set them to zero to avoid confusing drivers. Drivers will only check the flags by ANDing. Signed-off-by: Larry Finger Signed-off-by: Gabor Stefanik Signed-off-by: Michael Buesch Signed-off-by: John W. Linville --- drivers/ssb/pci.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/ssb/pci.c b/drivers/ssb/pci.c index 904b1a8d08859..57c4ccfab1ee7 100644 --- a/drivers/ssb/pci.c +++ b/drivers/ssb/pci.c @@ -484,6 +484,11 @@ static int sprom_extract(struct ssb_bus *bus, struct ssb_sprom *out, goto unsupported; } + if (out->boardflags_lo == 0xFFFF) + out->boardflags_lo = 0; /* per specs */ + if (out->boardflags_hi == 0xFFFF) + out->boardflags_hi = 0; /* per specs */ + return 0; unsupported: ssb_printk(KERN_WARNING PFX "Unsupported SPROM revision %d " -- 2.39.5