Bug 116344 - Incorrect config macro in tda1004x.c
Summary: Incorrect config macro in tda1004x.c
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: i686
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Arjan van de Ven
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2004-02-20 10:18 UTC by Neil Aspinall
Modified: 2007-11-30 22:10 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2004-05-03 19:02:09 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch (735 bytes, patch)
2004-02-24 12:35 UTC, Neil Aspinall
no flags Details | Diff

Description Neil Aspinall 2004-02-20 10:18:32 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6)
Gecko/20040210 Firefox/0.8

Description of problem:
The driver for the tda1004x DVB frontend
(drivers/media/dvb/frontends/tda1004x.c) contains references to a
macro DVB_TDA1004X_FIRMWARE_FILE.

This is not defined by the config/build process; the macro that
actually is defined is CONFIG_DVB_TDA1004X_FIRMWARE_FILE.

Version-Release number of selected component (if applicable):
kernel-2.6.2-1.87smp

How reproducible:
Always

Steps to Reproduce:
1. Unset CONFIG_STANDALONE and set CONFIG_DVB_TDA1004X=m and
CONFIG_DVB_TDA1004X_FIRMWARE_FILE=/some/path/to/firmware in your
.config (e.g. make xconfig; unset option to ignore drivers requiring
firmware; go to DVB section and enable tda1004x frontend)
2. Make, install and load tda1004x driver

    

Actual Results:  dmesg shows that the driver has been unable to load
firmware from "/usr/lib/hotplug/firmware/tda1004x.mc"; it has ignored
whatever path you defined and used its default instead

Expected Results:  Driver should load firmware from /some/path/to/firmware

Additional info:

Comment 1 Neil Aspinall 2004-02-24 12:35:58 UTC
Created attachment 97986 [details]
Patch

Comment 2 Dave Jones 2004-02-24 13:15:58 UTC
NACK.  This is finally being fixed properly upstream by using the
firmware loader interface.



Note You need to log in before you can comment on or make changes to this bug.