Re: [PATCH v3 1/2] staging: skein: Adds CryptoAPI Support

2014-10-23 Thread Dan Carpenter
First of all this patch does too many things at once. Collapsing files and deleting them needs to be done in separate patches from adding them. There is the "one thing per patch" and each one of those is a separate thing. On Thu, Oct 23, 2014 at 05:12:10PM -0500, Eric Rost wrote: > +int

Re: [PATCH v3 1/2] staging: skein: Adds CryptoAPI Support

2014-10-23 Thread Dan Carpenter
First of all this patch does too many things at once. Collapsing files and deleting them needs to be done in separate patches from adding them. There is the one thing per patch and each one of those is a separate thing. On Thu, Oct 23, 2014 at 05:12:10PM -0500, Eric Rost wrote: +int