Re: [PATCH] qlge: Fix incorrect usage of module parameters andnetdev msg level

From: Ron Mercer
Date: Thu Nov 18 2010 - 17:29:42 EST


On Thu, Nov 18, 2010 at 01:50:02PM -0800, Sonny Rao wrote:
> Driver appears to be mistaking the permission field with default value
> in the case of debug and qlge_irq_type.
>
> Driver is also passing debug as a bitmask into netif_msg_init()
> which wants a number of bits. Ron Mercer suggests we should
> change this to pass in -1 so the defaults get used instead,
> which makes the default much less verbose.
>
> Signed-off-by: Milton Miller <miltonm@xxxxxxx>
> Signed-off-by: Sonny Rao <sonnyrao@xxxxxxxxxxxxxxxxxx>

Milton/Sonny,
Did you happen to test this to see if the log output was the same?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/