Making it an option looks like a reasonable idea.
However, could you mind the indentation style of the driver, please?
On Mon, Aug 06, 2001 at 05:48:04PM +0200, Martin Knoblauch wrote:
> after realizing that my first attempt for this patch was to
> enthusiastic, I have no a somewhat stripped down version. Compiles
> against 2.4.7-ac7.
> The patch adds the option "power_save" to eepro100. If "1" (default),
> power save handling is done as normal. If "0", no power saving is done.
> This is to workaround some flaky eepro100 adapters that do not survive
> D0->D2-D0 transitions.
> @@ -1833,7 +1842,8 @@
> if (speedo_debug > 0)
> printk(KERN_DEBUG "%s: %d multicast blocks dropped.\n", dev->name, i);
> - pci_set_power_state(sp->pdev, 2);
> + if (power_save)
> + pci_set_power_state(sp->pdev, 2);
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to email@example.com
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/