Re: [PATCH 1/1] sep: Introduce sep driver

[Greg KH <greg@kroah.com>]

On Wed, Nov 17, 2010 at 03:45:36PM -0800, Mark Allyn wrote:
> This driver is for the Security Processor, a dedicated encryption
> and decryption driver that is used on the Intel mobile platform.
> 
> This has been checked with checkpatch and there are four
> warnings for lines over 80 charactors.
> 
> There is one compile warning. This is for a function that is
> only used if the rar register driver is needed. There is an
> ifdef in a header file that stubs out the rar register driver
> if the rar register is not configured.
> 
> This patch is referenced off of next-20101117.
> 

Minor comment:

> --- /dev/null
> +++ b/drivers/staging/sep/TODO
> @@ -0,0 +1,12 @@
> +Todo's so far (from Alan Cox)
> +- Fix firmware loading - Done 09/10 M. Allyn
> +- Get firmware into firmware git tree - Firmware is non open source
> +- Review and tidy each algorithm function - Done 09/10 M. Allyn
> +- Check whether it can be plugged into any of the kernel crypto API
> +  interfaces - Crypto API 'glue' is still not ready to submit
> +- Do something about the magic shared memory interface and replace it
> +  with something saner (in Linux terms) - Done 09/10 M. Allyn
> +- Clean up unused ioctls - Needs vendor help
> +- Clean up unused fields in ioctl structures - Needs vendor help
> +- 64 bit size to be used for all user space addresses passed
> +  to ioctl - Done 10/10 M. Allyn

Remove entries in the TODO file that you have finished, only list what
needs to be done.

Also, please put your email address in the file for who to send patches
to for this driver, like the other TODO files in the
drivers/staging/*/TODO files have.

Care to send a follow-on patch for these minor changes?

thanks,

greg k-h