Re: [2.6 patch] drivers/ieee1394/ieee1394_transactions.c: possible cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 19, 2005 at 02:45:23AM +0200, Adrian Bunk wrote:
> This patch contains the following possible cleanups:
> - #if 0 the following unused global functions:
>   - hpsb_lock
>   - hpsb_send_gasp
> - ieee1394_transactions.h: remove the stale hpsb_lock64 prototype

I also removed the EXPORT_SYMBOL of hpsb_lock, since we're not (yet?)
accepting your earlier patch.

Applied to our SVN and queued for 2.6.13.  Thanks.

Jody

> Signed-off-by: Adrian Bunk <[email protected]>
> 
> ---
> 
>  drivers/ieee1394/ieee1394_transactions.c |    3 +++
>  drivers/ieee1394/ieee1394_transactions.h |    7 -------
>  2 files changed, 3 insertions(+), 7 deletions(-)
> 
> --- linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_transactions.h.old	2005-04-19 00:24:13.000000000 +0200
> +++ linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_transactions.h	2005-04-19 00:25:00.000000000 +0200
> @@ -53,12 +53,5 @@
>  	      u64 addr, quadlet_t *buffer, size_t length);
>  int hpsb_write(struct hpsb_host *host, nodeid_t node, unsigned int generation,
>  	       u64 addr, quadlet_t *buffer, size_t length);
> -int hpsb_lock(struct hpsb_host *host, nodeid_t node, unsigned int generation,
> -	      u64 addr, int extcode, quadlet_t *data, quadlet_t arg);
> -int hpsb_lock64(struct hpsb_host *host, nodeid_t node, unsigned int generation,
> -		u64 addr, int extcode, octlet_t *data, octlet_t arg);
> -int hpsb_send_gasp(struct hpsb_host *host, int channel, unsigned int generation,
> -                   quadlet_t *buffer, size_t length, u32 specifier_id,
> -                   unsigned int version);
>  
>  #endif /* _IEEE1394_TRANSACTIONS_H */
> --- linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_transactions.c.old	2005-04-19 00:24:31.000000000 +0200
> +++ linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_transactions.c	2005-04-19 00:24:51.000000000 +0200
> @@ -535,6 +535,7 @@
>          return retval;
>  }
>  
> +#if 0
>  
>  int hpsb_lock(struct hpsb_host *host, nodeid_t node, unsigned int generation,
>  	      u64 addr, int extcode, quadlet_t *data, quadlet_t arg)
> @@ -599,3 +600,5 @@
>  
>  	return retval;
>  }
> +
> +#endif  /*  0  */
> 
> 
> 
> -------------------------------------------------------
> This SF.Net email is sponsored by: New Crystal Reports XI.
> Version 11 adds new functionality designed to reduce time involved in
> creating, integrating, and deploying reporting solutions. Free runtime info,
> new features, or free trial, at: http://www.businessobjects.com/devxi/728
> _______________________________________________
> mailing list [email protected]
> https://lists.sourceforge.net/lists/listinfo/linux1394-devel

-- 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux