Re: RFC: [PATCH 2/5] staging: sep: update existing sep headers

[Greg KH <greg@kroah.com>]

On Sun, Feb 27, 2011 at 09:11:09PM -0800, Mark Allyn wrote:
> Updates include new structures and constants necessary
> for kernel crypto operation
> 
> Some functions in sep_main.c are made available to sep_crypto.c
> which will be added in subsequent patch
> 
> Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
> ---
>  drivers/staging/sep/sep_dev.h           |   24 ++++-
>  drivers/staging/sep/sep_driver_api.h    |  192 ++++++++++++++++++++++++++++++-
>  drivers/staging/sep/sep_driver_config.h |   14 +-
>  3 files changed, 215 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/staging/sep/sep_dev.h b/drivers/staging/sep/sep_dev.h
> index 0ffe68c..122dc94 100644
> --- a/drivers/staging/sep/sep_dev.h
> +++ b/drivers/staging/sep/sep_dev.h
> @@ -5,8 +5,8 @@
>   *
>   *  sep_dev.h - Security Processor Device Structures
>   *
> - *  Copyright(c) 2009,2010 Intel Corporation. All rights reserved.
> - *  Contributions(c) 2009,2010 Discretix. All rights reserved.
> + *  Copyright(c) 2009,2011 Intel Corporation. All rights reserved.
> + *  Contributions(c) 2009,2011 Discretix. All rights reserved.

So the contributions came from both companies?  Care to get a
signed-off-by from Discretix as well?

>   *
>   *  This program is free software; you can redistribute it and/or modify it
>   *  under the terms of the GNU General Public License as published by the Free
> @@ -28,6 +28,7 @@
>   *
>   *  CHANGES
>   *  2010.09.14  upgrade to Medfield
> + *  2011.02.22  enable kernel crypto

Changes do NOT belong in the individual files, that's what the git
changelog is for.

thanks,

greg k-h