On Thu, Jul 06, 2006 at 03:51:11PM +0200, Johannes Berg wrote:
> On Thu, 2006-07-06 at 15:45 +0200, Martin Langer wrote:
> Also, I don't see why you have two arrays. Why not just a single array
> with
>
> enum bcm_fw_type {
> BCM_FW_NO_MORE,
> BCM_FW_PCM5,
> ...
> };
>
> struct bcm_fw_revision {
> enum bcm_fw_type type;
> u8[16] md5sum;
> u8 maj,min,rc,patchlevel;
> };
There's a general firmware part which returns a s32 version number. This
can be used by all drivers (not only bcm43xx!!). And the driver specific
part transforms s32 into the broadcom number. That part is completely
placed in the bcm43xx code. Was that your problem? I guess I
misunderstood it in the former mail.
The real problem is that each company has it's own version scheme.
That's why I took that transformation. I don't want to put
maj,min,rc,patchlevel into the firmware_class.c file.
Martin
_______________________________________________
Bcm43xx-dev mailing list
[email protected]
http://lists.berlios.de/mailman/listinfo/bcm43xx-dev
