On Wed, 14 May 2008, Pavel Roskin wrote:
On Tue, 2008-05-13 at 17:59 -0700, John Daiker wrote:
Hey All,
My first patch for the mrv8k driver. My machine locks up when I try to
load a firmware, so I'm not sure if this patch helps anybody or not.
I don't see any changes for mrv_reg_read32() claimed in the description.
He is checking the return value to 0x5 instead of 5.
In fact, I don't see any changes to the code except comments and
messages. I don't see how it can help against lockups, unless you
missed some part of the patch.
It does not. But I pushed the comment and indent fixes to mrv8k
nevertheless.
You may want to spell check your patch, including the description,
especially if the patch is fixing spelling mistakes made by others.
--
Regards,
Pavel Roskin
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html