Bug 227437 - iwn 6235 0x5260: sub-id for this card is not supported, panics if patch added
Summary: iwn 6235 0x5260: sub-id for this card is not supported, panics if patch added
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: wireless (show other bugs)
Version: CURRENT
Hardware: amd64 Any
: --- Affects Some People
Assignee: freebsd-wireless (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-11 02:47 UTC by Andy
Modified: 2019-02-12 23:44 UTC (History)
2 users (show)

See Also:


Attachments
the list of iwn errors dumped to dmesg/console (7.34 KB, text/plain)
2018-04-11 02:47 UTC, Andy
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andy 2018-04-11 02:47:06 UTC
Created attachment 192426 [details]
the list of iwn errors dumped to dmesg/console

Hi, I recently purchased a lenovo-"branded" (has a lenovo fru part number) 6235 wireless card for my Thinkpad T440p.  This was a downgrade from the stock 7260 in the hopes that iwn support would be better than iwm.  However, the sub-id for this card is not supported.  This is the same card that the original poster in this thread had:
https://lists.freebsd.org/pipermail/freebsd-wireless/2014-December/005262.html

I added the sub-id, recompiled, and the card seems to work, but the driver panics and restarts.  I'm back on the 7260 but I'm happy to test any proposed fixes.  The change I made isn't patch-worthy, I just added a new line in if_iwn_devid.h:
+#define	IWN_SDID_6035_5		0x5260
and add a case in if_iwn.c for that ID.

The error message that was logged is attached, also this is the detection line from dmesg:
iwn0: <Intel Centrino Advanced 6235> mem 0xf0400000-0xf0401fff irq 17 at device 0.0 on pci
iwn0: iwn_read_firmware: ucode rev=0x12a80601
Comment 1 Andriy Voskoboinyk freebsd_committer freebsd_triage 2019-01-25 10:33:37 UTC
Hi,

The subid was added in base r335812; is it still unusable on recent HEAD?
Comment 2 Andy 2019-02-12 23:44:16 UTC
I can confirm that on -current (I have a freshly compiled r344052), this card works.  In fact, I'm using it to fill in this response.  I'll see if any issues crop up, but I see no reason not to close this.