Re: [PATCH] some network __init code

[Jeff Garzik]

> 1. What are the char* -> char array conversions of "version" strings for ?

char*="blah" generates a char pointer variable, pointing to the constant
string "blah".  char[]="blah" eliminates the char pointer variable, so
the resulting code is [slightly] smaller.

> 3. The following patch
>    - marks most of the version strings __initdata/__devinitdata (necessary
>      removing of "const" from their declaration), removes unnecessary format
>      strings from their printk()s, moves to __init/adds log level markers to
>      them (KERN_*)
>    - adds/fixes some other __init code,
>    - removes some unnecessary zero initializers
>    from most of the network drivers.

looks ok at a glance, I will probably apply it after reviewing further.

note a further cleanup is to look at each driver, and make sure (a) it
-always- printk's version if -DMODULE, and (b) if only printk's version
if hardware is found, if not -DMODULE.  You can look at pci net drivers
in 2.4.4-pre6 for an example of how I did this.

-- 
Jeff Garzik      | The difference between America and England is that
Building 1024    | the English think 100 miles is a long distance and
MandrakeSoft     | the Americans think 100 years is a long time.
                 |      (random fortune)