0004734: mdev doesn't work as firmware loader, patch attached

BusyBox Bug and Patch Tracking

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0004734

[BusyBox] Kernel Module Support

minor

always

08-26-08 01:45

08-29-08 01:56

Reporter

hughdavenport

View Status

public

Assigned To

BusyBox

Priority

normal

Resolution

unable to reproduce

Status

closed

Product Version

1.12.x

Summary

0004734: mdev doesn't work as firmware loader, patch attached

Description

using mdev as firmware loader didn't work for me so I investigated the code
fix was simple, close to end of mdev.c if had
if (ENABLE...LOAD_FIRMWARE) {
...
}
I just changed it to use a preprocessor #if. this has fixed it, patch is supplied

Additional Information

Attached Files

busybox-1.12.0.patch [^] (533 bytes) 08-26-08 01:45

Relationships

Notes
(0010874) vda 08-29-08 01:23	I do not understand. - if (ENABLE_FEATURE_MDEV_LOAD_FIRMWARE) { +#if ENABLE_FEATURE_MDEV_LOAD_FIRMWARE So, it was not entering the if() even though ENABLE_FEATURE_MDEV_LOAD_FIRMWARE == 1? Wow.

(0010894) hughdavenport 08-29-08 01:43	yeh thats what i though. patch seemed to fix it, but whats stranger is that it now works without the patch. oh well, i guess this issue can be dropped. must of been something else to do with my system that has since been fixed. cheers

Issue History
Date Modified	Username	Field	Change
08-26-08 01:45	hughdavenport	New Issue
08-26-08 01:45	hughdavenport	Status	new => assigned
08-26-08 01:45	hughdavenport	Assigned To	=> BusyBox
08-26-08 01:45	hughdavenport	File Added: busybox-1.12.0.patch
08-29-08 01:23	vda	Note Added: 0010874
08-29-08 01:43	hughdavenport	Note Added: 0010894
08-29-08 01:56	bernhardf	Status	assigned => closed
08-29-08 01:56	bernhardf	Resolution	open => unable to reproduce