[PATCH] remove incorrect and redundant modprobe modalias rule

[Colin Watson]

The following patch removes the udev rule that does '/sbin/modprobe
$modalias', because (a) udev has no $modalias variable (it should be
$env{MODALIAS} instead) and (b) udev installations these days generally
have a rule in the standard ruleset that does this already, and
duplicating the rule just results in an unnecessary extra invocation of
modprobe.

If you don't want to remove the rule, then please correct $modalias to
$env{modalias}.

--- udev/60-pcmcia.rules
+++ udev/60-pcmcia.rules
@@ -1,9 +1,5 @@
 # PCMCIA devices:
 #
-# modprobe $modalias loads all possibly appropriate modules
-ACTION=="add", SUBSYSTEM=="pcmcia", MODALIAS=="*", \
-		RUN+="/sbin/modprobe $modalias"
-
 # Very few CIS firmware entries (which we use for matching)
 # are so broken that we need to read out random bytes of it
 # instead of the manufactor, card or product ID. Then the

--- udev/60-pcmcia.rules.static
+++ udev/60-pcmcia.rules.static
@@ -1,9 +1,5 @@
 # PCMCIA devices:
 #
-# modprobe $modalias loads all possibly appropriate modules
-ACTION=="add", SUBSYSTEM=="pcmcia", MODALIAS=="*", \
-		RUN+="/sbin/modprobe $modalias"
-
 # Very few CIS firmware entries (which we use for matching)
 # are so broken that we need to read out random bytes of it
 # instead of the manufactor, card or product ID. Then the


Thanks,

-- 
Colin Watson                                       [cjwatson at ubuntu.com]