Bug 110376 - [patch] add some more explanations for the iwi/ipw firmware inclusion
Summary: [patch] add some more explanations for the iwi/ipw firmware inclusion
Status: Closed FIXED
Alias: None
Product: Documentation
Classification: Unclassified
Component: Manual Pages (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-doc (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2007-03-16 07:10 UTC by Eygene Ryabinkin
Modified: 2019-01-31 11:02 UTC (History)
3 users (show)

See Also:


Attachments
file.diff (523 bytes, patch)
2007-03-16 07:10 UTC, Eygene Ryabinkin
no flags Details | Diff
file.diff (617 bytes, patch)
2007-03-16 07:10 UTC, Eygene Ryabinkin
no flags Details | Diff
file.diff (617 bytes, patch)
2007-03-16 07:10 UTC, Eygene Ryabinkin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eygene Ryabinkin 2007-03-16 07:10:06 UTC
The inclusion of iwi/ipw firmware into the base system was a great
step. But the UPDATING and the iwi/ipw manual pages are saying
nothing about the need of inclusion of the iwifw/ipwfw modules
into the kernel configuration. And some people are building just
the restricted set of kernel modules, so they should know what they
will need to add to enable the system's firmware modules to be
built and installed into a proper place.

Also, manual pages iwi(4) and ipw(4) are specifying that the
license kenv tunable is license.intel_<name>.license_ack. Reading
the /sys/tools/fw_stub.awk I had a strong feeling that the right
name is license.<name>.license_ack as noted in the original commit
message and in the UPDATING.

Fix: The following patches will do the trick.
How-To-Repeat: Look into the UPDATING and iwi/ipw manual pages: they are telling
nothing about the need of iwifw/ipwfw kernel modules. And the man
pages are telling about license.intel_{iwi,ipw}.license_ack.
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2007-03-16 07:58:36 UTC
Responsible Changed
From-To: freebsd-bugs->freebsd-doc

Reclassify.
Comment 2 Szilveszter Adam 2007-03-16 09:21:21 UTC
Hello,

On Fri, Mar 16, 2007 at 10:09:18AM +0300, Eygene Ryabinkin wrote:
> Also, manual pages iwi(4) and ipw(4) are specifying that the
> license kenv tunable is license.intel_<name>.license_ack. Reading
> the /sys/tools/fw_stub.awk I had a strong feeling that the right
> name is license.<name>.license_ack as noted in the original commit
> message and in the UPDATING.

Well, I also had this feeling at first (especially since I tend to trust
UPDATING in these matters) but it is not so. I know this because I have
tried the variant you suggested first and found out fairly quickly that
the firmware did not load. :-) This was on iwi, btw, but I do not think
this matters. With the variant in the man page, it works 1A. I specify
the var in /boot/loader.conf. And the var name starts with "legal" not
"license" in both cases.

Your first point about required kernel modules stands however :-)

-- 
Regards:

Szilveszter ADAM
Budapest
Hungary
Comment 3 Warren Block 2010-01-03 22:47:52 UTC
ipw and iwi now use firmware(9) and so references to ifwfw(4) are 
obsolete:

   "This driver requires the firmware built with the ipwfw module to
    work."

This sentence can be eliminated.

   "For the loaded firmware to be enabled for use the license at"

Should probably be changed to

   "For the IBM firmware to be enabled for use the license at"

Also, the references to ipwfw(4) in the SEE ALSO section should be 
removed.

-Warren Block * Rapid City, South Dakota USA
Comment 4 Eitan Adler freebsd_committer freebsd_triage 2017-12-31 07:59:54 UTC
For bugs matching the following criteria:

Status: In Progress Changed: (is less than) 2014-06-01

Reset to default assignee and clear in-progress tags.

Mail being skipped
Comment 5 Andriy Voskoboinyk freebsd_committer freebsd_triage 2019-01-31 11:02:58 UTC
Both man pages have firmware-related parts since base r178354; the tunable name is set to legal.intel_{iwi,ipw}.license_ack (just checked with ipw_bss module - it requires 'intel_ipw' license agreement).