Re: [Outreachy kernel] [PATCH 3/3] Staging: lustre: Remove unused macros

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Oct 21, 2015 at 11:21:53PM +0200, Ksenija Stanojevic wrote:
> Macros LPROCFS_CLIMP_CHECK and LPROCFS_CLIMP_EXIT are not used anymore,
> therefore remove them along with the comment.
> 
> Signed-off-by: Ksenija Stanojevic <ksenija.stanojevic@gmail.com>
> ---
>  drivers/staging/lustre/lustre/include/lprocfs_status.h | 13 -------------
>  1 file changed, 13 deletions(-)
> 
> diff --git a/drivers/staging/lustre/lustre/include/lprocfs_status.h b/drivers/staging/lustre/lustre/include/lprocfs_status.h
> index 6e41129..532d195 100644
> --- a/drivers/staging/lustre/lustre/include/lprocfs_status.h
> +++ b/drivers/staging/lustre/lustre/include/lprocfs_status.h
> @@ -626,19 +626,6 @@ void lprocfs_stats_collect(struct lprocfs_stats *stats, int idx,
>  int lprocfs_single_release(struct inode *, struct file *);
>  int lprocfs_seq_release(struct inode *, struct file *);
>  
> -/* You must use these macros when you want to refer to
> - * the import in a client obd_device for a lprocfs entry */
> -#define LPROCFS_CLIMP_CHECK(obd) do {	   \
> -	typecheck(struct obd_device *, obd);    \

Note, when you turn this into an inline function, you don't need the
typecheck() call anymore, right?

thanks,

greg k-h