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

From: Jody McIntyre
Date: Thu Apr 21 2005 - 15:53:52 EST


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 <bunk@xxxxxxxxx>
>
> ---
>
> 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 linux1394-devel@xxxxxxxxxxxxxxxxxxxxx
> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/