Skip to contentSkip to conversation

ssb: Do not disable PCI host on non-Mips

PATCH In review Sven Joachim 3 participants 7 messages Jan 26, 2018

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ssb: Do not disable PCI host on non-Mips

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")
---
 drivers/ssb/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/ssb/Kconfig +1 -1
index 71c73766ee22..65af12c3bdb2 100644
@@ -32,7 +32,7 @@ config SSB_BLOCKIO
 
 config SSB_PCIHOST_POSSIBLE
 	bool
-	depends on SSB && (PCI = y || PCI = SSB) && PCI_DRIVERS_LEGACY
+	depends on SSB && (PCI = y || PCI = SSB) && (PCI_DRIVERS_LEGACY || !MIPS)
 	default y
 
 config SSB_PCIHOST
Signature · 1 line
2.15.1
Reviewed-by on v1 1/1
#1 lore raw
Quoted 8 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")

Whoops, thats a very good point. I hadn't twigged that PCI_DRIVERS_LEGACY was MIPS specific (one of the disadvantages of using "tig grep" I suppose!).

I think this is obviously correct, so it'd be great to squeeze it into 4.15 final.

Fortunately the other related change, commit 664eadd6f44b ("bcma: Fix 'allmodconfig' and BCMA builds on MIPS targets"), already depends on MIPS so doesn't have the same issue.

Sorry for the breakage!
James

Quoted 20 lines
---
 drivers/ssb/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/ssb/Kconfig +1 -1
index 71c73766ee22..65af12c3bdb2 100644
@@ -32,7 +32,7 @@ config SSB_BLOCKIO
 
 config SSB_PCIHOST_POSSIBLE
 	bool
-	depends on SSB && (PCI = y || PCI = SSB) && PCI_DRIVERS_LEGACY
+	depends on SSB && (PCI = y || PCI = SSB) && (PCI_DRIVERS_LEGACY || !MIPS)
 	default y
 
 config SSB_PCIHOST
Signature · 1 line
2.15.1
#2 lore raw
Quoted 10 lines · James Hogan
Quoted 8 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")
Quoted 8 lines

Whoops, thats a very good point. I hadn't twigged that PCI_DRIVERS_LEGACY was MIPS specific (one of the disadvantages of using "tig grep" I suppose!).

I think this is obviously correct, so it'd be great to squeeze it into 4.15 final.

I'm not sure if I'm able to get it to 4.15 as it has go via the net tree, and we have only two days before the (likely) final release, but I'll try.

Signature · 1 line
Kalle Valo
#3 lore raw
Quoted 16 lines · Kalle Valo
Quoted 14 lines · James Hogan
Quoted 8 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")

Whoops, thats a very good point. I hadn't twigged that PCI_DRIVERS_LEGACY was MIPS specific (one of the disadvantages of using "tig grep" I suppose!).

Quoted 8 lines
Quoted 4 lines

I think this is obviously correct, so it'd be great to squeeze it into 4.15 final.

I'm not sure if I'm able to get it to 4.15 as it has go via the net tree, and we have only two days before the (likely) final release, but I'll try.

Too late, Dave already sent his last pull request for 4.15. This will be in 4.16.

Signature · 1 line
Kalle Valo
#4 lore raw
Quoted 23 lines · Kalle Valo
Quoted 21 lines · Kalle Valo
Quoted 18 lines · James Hogan
Quoted 8 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")

Whoops, thats a very good point. I hadn't twigged that PCI_DRIVERS_LEGACY was MIPS specific (one of the disadvantages of using "tig grep" I suppose!).

I think this is obviously correct, so it'd be great to squeeze it into 4.15 final.

Quoted 6 lines
Quoted 3 lines

I'm not sure if I'm able to get it to 4.15 as it has go via the net tree, and we have only two days before the (likely) final release, but I'll try.

Too late, Dave already sent his last pull request for 4.15. This will be in 4.16.

Okay, thanks anyway (and sorry again!).

Cheers
James

#5 lore raw
In reply to Sven Joachim
Quoted 9 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")

Patch applied to wireless-drivers.git, thanks.

a9e6d44ddecc ssb: Do not disable PCI host on non-Mips

Signature · 3 lines
https://patchwork.kernel.org/patch/10185397/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
#6 lore raw
In reply to Sven Joachim
Quoted 9 lines · Sven Joachim

After upgrading an old laptop to 4.15-rc9, I found that the eth0 and wlan0 interfaces had disappeared. It turns out that the b43 and b44 drivers require SSB_PCIHOST_POSSIBLE which depends on PCI_DRIVERS_LEGACY, a config option that only exists on Mips.

Fixes 58eae1416b80 ("ssb: Disable PCI host for PCI_DRIVERS_GENERIC")

Patch applied to wireless-drivers.git, thanks.

a9e6d44ddecc ssb: Do not disable PCI host on non-Mips

Signature · 3 lines
https://patchwork.kernel.org/patch/10185397/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches