[PATCH 01/22] staging: lustre: Remove all new typedefs in libcfs_hash.h

[Lisa Nguyen]

On Sun, Oct 20, 2013 at 8:23 PM, Greg KH <gregkh at linuxfoundation.org> wrote:
> On Sun, Oct 20, 2013 at 03:45:46PM -0700, Lisa Nguyen wrote:
>> Modified all structs in libcfs_hash header file to eliminate
>> the "Do not add new typedefs" warning generated by checkpatch.pl
>> and meet kernel coding style.
>>
>> Signed-off-by: Lisa Nguyen <lisa at xenapiadmin.com>
>
> This patch breaks the build so badly, I know you didn't test it alone :(
>
> Your follow-on patches do fix up the build breakage, which is nice, but
> you can never break the build with any individual patch, that's not how
> kernel development works, sorry.
>

Good to know. Something new I learned today and will know how to keep
maintainers happy :)

> So, can you change this to just fix one typedef at a time, for the whole
> chunk of code?  That should be managable, and reviewable, instead of
> having to put all 22 patches here into one big patch (which isn't ok.)
>
> thanks,
>
> greg k-h

I will do one typedef at a time and will resend for feedback.

Please ignore this patch series. I will create a new patchset to
modify one typedef at a time before I send more.

Thanks,
Lisa