On 07/26/2010 07:05 PM, Corey Minyard wrote:Sorry, you are right.
Please run this through checkpatch, as it has coding style violations.yhlu@linux-siqj:~/xx/xx/kernel/tip/linux-2.6> ./scripts/checkpatch.pl patches/ipmi_reg_size.patch
total: 0 errors, 0 warnings, 105 lines checked
patches/ipmi_reg_size.patch has no obvious style problems and is ready for submission.
yhlu@linux-siqj:~/xx/xx/kernel/tip/linux-2.6>
Ok, thanksAlso, this patch appears to fix bugs in addition to adding the print.in the comment log, i already mentioned that.
Can we have a separate patch for that?
will separate it to twol
Well, I'm not sure on this. You are right, it is printed for those paths and not for DMI or SPMI cases. Printing too much information is not generally a good idea, but this may be useful. I guess to make it consistent it would be best to add this.I'm also not clear on the reason for this. I believe all thisthen why there is printing for ACPI path and pci path?
information is already available in /proc/ipmi/<if#>/params. I don't
think there is a strong reason to print it to the log.