public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: linux1394-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Cc: Jody McIntyre <scjody@modernduck.com>, Adrian Bunk <bunk@stusta.de>
Subject: Re: [2.6 patch] drivers/ieee1394/: schedule unused EXPORT_SYMBOL's for removal (fwd)
Date: Thu, 07 Jul 2005 21:30:21 +0200	[thread overview]
Message-ID: <42CD82CD.5040903@s5r6.in-berlin.de> (raw)
In-Reply-To: <20050707144513.GG10001@conscoop.ottawa.on.ca>

Jody McIntyre wrote:
> On Mon, Jul 04, 2005 at 01:24:05AM +0200, Adrian Bunk wrote:
>>What shall I do?
>>- resend this patch with the removal date set to August or
> 
> That should be fine.  I'll watch for it this time.  August isn't much
> time though; isn't the "standard" 6 months to a year?

August is fine; it's our sloppiness that the remove announcement did not 
go in earlier. People who run external projects that use such symbols 
might have had forewarning by the previous discussion rounds on 
linux1394-devel.

>>+What:	remove the following ieee1394 EXPORT_SYMBOL's:
>>+	- hpsb_send_phy_config
>>+	- hpsb_send_packet_and_wait
>>+	- highlevel_add_host
>>+	- highlevel_remove_host
>>+	- nodemgr_for_each_host
>>+	- csr1212_create_csr
>>+	- csr1212_init_local_csr
>>+	- csr1212_new_immediate
>>+	- csr1212_associate_keyval
>>+	- csr1212_new_string_descriptor_leaf
>>+	- csr1212_destroy_csr
>>+	- csr1212_generate_csr_image
>>+	- csr1212_parse_csr

Now that we are at it, the following EXPORT_SYMBOLs should be removed too...
	_csr1212_read_keyval
	_csr1212_destroy_keyval

>>+Files:	drivers/ieee1394/ieee1394_core.c
>>+Why:	No modular usage in the kernel.

...at the same place and for the same reason.
-- 
Stefan Richter
-=====-=-=-= -=== --===
http://arcgraph.de/sr/

  reply	other threads:[~2005-07-07 19:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-03 23:24 [2.6 patch] drivers/ieee1394/: schedule unused EXPORT_SYMBOL's for removal (fwd) Adrian Bunk
2005-07-07 14:45 ` Jody McIntyre
2005-07-07 19:30   ` Stefan Richter [this message]
2005-07-09  3:07     ` [2.6 patch] drivers/ieee1394/: schedule unused EXPORT_SYMBOL's for removal Adrian Bunk
2005-07-09  7:22       ` Stefan Richter
2005-07-09  7:50         ` Ben Collins
2005-07-09 10:32           ` alternative [PATCH] 1/2) " Stefan Richter
2005-07-09 16:13             ` Ben Collins
2005-07-09 21:10             ` Ben Collins
2005-07-09 10:33           ` alternative [PATCH] 2/2) " Stefan Richter
2005-07-09 18:55           ` [2.6 patch] " Adrian Bunk
2005-07-09 19:43             ` Stefan Richter
2005-07-09 20:49             ` Ben Collins

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42CD82CD.5040903@s5r6.in-berlin.de \
    --to=stefanr@s5r6.in-berlin.de \
    --cc=bunk@stusta.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux1394-devel@lists.sourceforge.net \
    --cc=scjody@modernduck.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox