RE: [PATCH 2/2] Staging: hv: General maintenance of TODO file

From: Hank Janssen
Date: Thu Feb 11 2010 - 17:22:57 EST


>>
>> Removed two line;
>> Audit block driver
>> Audit scsi driver
>> They were added by us as part of the original code
>> Submission to make sure these drivers kept on working
>> Correctly with all the patches that came in from the
>> Community.
>
>Um, no, those things still need to be done.

What additional things are expected? We solved the original
reason we added the lines, so I want to make sure we do
additional work as is expected. We are going through general
code cleanup, and the TODO file now.

>> Added a new line to fix mostly insane camelcase naming
>> We are working on that right now, but it will take a bit
>> Due to the pervasive usage of it.
>
>That's fine, but it fits under the general "fix coding style issues".
>
>Hm, that isn't spelled out, but it is implicit, I doubt this really
>needs to be documented in the file, it's just common knowledge :)

Want me to add a proper line item for that? Or drop this patch
completely for the time being?

Thanks,

Hank.

--
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/