2005-04-17 19:57:42

by Adrian Bunk

[permalink] [raw]
Subject: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

This patch removes unneeded EXPORT_SYMBOL's.

Signed-off-by: Adrian Bunk <[email protected]>

---

drivers/ieee1394/ieee1394_core.c | 16 ----------------
1 files changed, 16 deletions(-)

--- linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_core.c.old 2005-04-17 20:49:31.000000000 +0200
+++ linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_core.c 2005-04-17 20:57:34.000000000 +0200
@@ -1225,9 +1225,7 @@
EXPORT_SYMBOL(hpsb_set_packet_complete_task);
EXPORT_SYMBOL(hpsb_alloc_packet);
EXPORT_SYMBOL(hpsb_free_packet);
-EXPORT_SYMBOL(hpsb_send_phy_config);
EXPORT_SYMBOL(hpsb_send_packet);
-EXPORT_SYMBOL(hpsb_send_packet_and_wait);
EXPORT_SYMBOL(hpsb_reset_bus);
EXPORT_SYMBOL(hpsb_bus_reset);
EXPORT_SYMBOL(hpsb_selfid_received);
@@ -1246,9 +1244,6 @@
EXPORT_SYMBOL(hpsb_make_lock64packet);
EXPORT_SYMBOL(hpsb_make_phypacket);
EXPORT_SYMBOL(hpsb_make_isopacket);
-EXPORT_SYMBOL(hpsb_read);
-EXPORT_SYMBOL(hpsb_write);
-EXPORT_SYMBOL(hpsb_lock);
EXPORT_SYMBOL(hpsb_packet_success);

/** highlevel.c **/
@@ -1265,8 +1260,6 @@
EXPORT_SYMBOL(hpsb_set_hostinfo_key);
EXPORT_SYMBOL(hpsb_get_hostinfo_bykey);
EXPORT_SYMBOL(hpsb_set_hostinfo);
-EXPORT_SYMBOL(highlevel_add_host);
-EXPORT_SYMBOL(highlevel_remove_host);
EXPORT_SYMBOL(highlevel_host_reset);

/** nodemgr.c **/
@@ -1275,7 +1268,6 @@
EXPORT_SYMBOL(hpsb_register_protocol);
EXPORT_SYMBOL(hpsb_unregister_protocol);
EXPORT_SYMBOL(ieee1394_bus_type);
-EXPORT_SYMBOL(nodemgr_for_each_host);

/** csr.c **/
EXPORT_SYMBOL(hpsb_update_config_rom);
@@ -1312,19 +1304,11 @@
EXPORT_SYMBOL(hpsb_iso_recv_flush);

/** csr1212.c **/
-EXPORT_SYMBOL(csr1212_create_csr);
-EXPORT_SYMBOL(csr1212_init_local_csr);
-EXPORT_SYMBOL(csr1212_new_immediate);
EXPORT_SYMBOL(csr1212_new_directory);
-EXPORT_SYMBOL(csr1212_associate_keyval);
EXPORT_SYMBOL(csr1212_attach_keyval_to_directory);
-EXPORT_SYMBOL(csr1212_new_string_descriptor_leaf);
EXPORT_SYMBOL(csr1212_detach_keyval_from_directory);
EXPORT_SYMBOL(csr1212_release_keyval);
-EXPORT_SYMBOL(csr1212_destroy_csr);
EXPORT_SYMBOL(csr1212_read);
-EXPORT_SYMBOL(csr1212_generate_csr_image);
EXPORT_SYMBOL(csr1212_parse_keyval);
-EXPORT_SYMBOL(csr1212_parse_csr);
EXPORT_SYMBOL(_csr1212_read_keyval);
EXPORT_SYMBOL(_csr1212_destroy_keyval);


2005-04-19 19:18:37

by Jody McIntyre

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Sun, Apr 17, 2005 at 09:57:07PM +0200, Adrian Bunk wrote:
> This patch removes unneeded EXPORT_SYMBOL's.

Didn't you already send something like this (with fewer removals, mind
you) in December?

http://marc.theaimsgroup.com/?l=linux1394-devel&m=110350765817261&w=2

Given the objections to your December patch, why should we accept this
one now?

Jody

>
> Signed-off-by: Adrian Bunk <[email protected]>
>
> ---
>
> drivers/ieee1394/ieee1394_core.c | 16 ----------------
> 1 files changed, 16 deletions(-)
>
> --- linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_core.c.old 2005-04-17 20:49:31.000000000 +0200
> +++ linux-2.6.12-rc2-mm3-full/drivers/ieee1394/ieee1394_core.c 2005-04-17 20:57:34.000000000 +0200
> @@ -1225,9 +1225,7 @@
> EXPORT_SYMBOL(hpsb_set_packet_complete_task);
> EXPORT_SYMBOL(hpsb_alloc_packet);
> EXPORT_SYMBOL(hpsb_free_packet);
> -EXPORT_SYMBOL(hpsb_send_phy_config);
> EXPORT_SYMBOL(hpsb_send_packet);
> -EXPORT_SYMBOL(hpsb_send_packet_and_wait);
> EXPORT_SYMBOL(hpsb_reset_bus);
> EXPORT_SYMBOL(hpsb_bus_reset);
> EXPORT_SYMBOL(hpsb_selfid_received);
> @@ -1246,9 +1244,6 @@
> EXPORT_SYMBOL(hpsb_make_lock64packet);
> EXPORT_SYMBOL(hpsb_make_phypacket);
> EXPORT_SYMBOL(hpsb_make_isopacket);
> -EXPORT_SYMBOL(hpsb_read);
> -EXPORT_SYMBOL(hpsb_write);
> -EXPORT_SYMBOL(hpsb_lock);
> EXPORT_SYMBOL(hpsb_packet_success);
>
> /** highlevel.c **/
> @@ -1265,8 +1260,6 @@
> EXPORT_SYMBOL(hpsb_set_hostinfo_key);
> EXPORT_SYMBOL(hpsb_get_hostinfo_bykey);
> EXPORT_SYMBOL(hpsb_set_hostinfo);
> -EXPORT_SYMBOL(highlevel_add_host);
> -EXPORT_SYMBOL(highlevel_remove_host);
> EXPORT_SYMBOL(highlevel_host_reset);
>
> /** nodemgr.c **/
> @@ -1275,7 +1268,6 @@
> EXPORT_SYMBOL(hpsb_register_protocol);
> EXPORT_SYMBOL(hpsb_unregister_protocol);
> EXPORT_SYMBOL(ieee1394_bus_type);
> -EXPORT_SYMBOL(nodemgr_for_each_host);
>
> /** csr.c **/
> EXPORT_SYMBOL(hpsb_update_config_rom);
> @@ -1312,19 +1304,11 @@
> EXPORT_SYMBOL(hpsb_iso_recv_flush);
>
> /** csr1212.c **/
> -EXPORT_SYMBOL(csr1212_create_csr);
> -EXPORT_SYMBOL(csr1212_init_local_csr);
> -EXPORT_SYMBOL(csr1212_new_immediate);
> EXPORT_SYMBOL(csr1212_new_directory);
> -EXPORT_SYMBOL(csr1212_associate_keyval);
> EXPORT_SYMBOL(csr1212_attach_keyval_to_directory);
> -EXPORT_SYMBOL(csr1212_new_string_descriptor_leaf);
> EXPORT_SYMBOL(csr1212_detach_keyval_from_directory);
> EXPORT_SYMBOL(csr1212_release_keyval);
> -EXPORT_SYMBOL(csr1212_destroy_csr);
> EXPORT_SYMBOL(csr1212_read);
> -EXPORT_SYMBOL(csr1212_generate_csr_image);
> EXPORT_SYMBOL(csr1212_parse_keyval);
> -EXPORT_SYMBOL(csr1212_parse_csr);
> EXPORT_SYMBOL(_csr1212_read_keyval);
> EXPORT_SYMBOL(_csr1212_destroy_keyval);
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> mailing list [email protected]
> https://lists.sourceforge.net/lists/listinfo/linux1394-devel

--

2005-04-19 19:44:10

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Tue, 2005-04-19 at 15:13 -0400, Jody McIntyre wrote:
> On Sun, Apr 17, 2005 at 09:57:07PM +0200, Adrian Bunk wrote:
> > This patch removes unneeded EXPORT_SYMBOL's.
>
> Didn't you already send something like this (with fewer removals, mind
> you) in December?
>
> http://marc.theaimsgroup.com/?l=linux1394-devel&m=110350765817261&w=2
>
> Given the objections to your December patch, why should we accept this
> one now?

since there still isn't a user ??


2005-04-19 22:17:46

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Arjan van de Ven wrote:
> On Tue, 2005-04-19 at 15:13 -0400, Jody McIntyre wrote:
>> On Sun, Apr 17, 2005 at 09:57:07PM +0200, Adrian Bunk wrote:
>> > This patch removes unneeded EXPORT_SYMBOL's.
...
>> Given the objections to your December patch, why should we accept this
>> one now?
>
> since there still isn't a user ??

There are users (though not in "the" kernel at the moment), have been
users, will be users.

Note: There is no architectural problem or something like that which
would be a reason to change the API.
--
Stefan Richter
-=====-=-=-= --== -===-
http://arcgraph.de/sr/

2005-04-20 07:26:37

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, 2005-04-20 at 00:00 +0200, Stefan Richter wrote:
> Arjan van de Ven wrote:
> > On Tue, 2005-04-19 at 15:13 -0400, Jody McIntyre wrote:
> >> On Sun, Apr 17, 2005 at 09:57:07PM +0200, Adrian Bunk wrote:
> >> > This patch removes unneeded EXPORT_SYMBOL's.
> ...
> >> Given the objections to your December patch, why should we accept this
> >> one now?
> >
> > since there still isn't a user ??
>
> There are users (though not in "the" kernel at the moment)

nor for the last 5 months... how long will it be ?

2005-04-20 16:32:11

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Arjan van de Ven wrote:
> On Wed, 2005-04-20 at 00:00 +0200, Stefan Richter wrote:
>>There are users (though not in "the" kernel at the moment)
> nor for the last 5 months... how long will it be ?

Have there been problems with the API during the past 5 months, except
that several kernel trees are using some parts of the API? (We are
actually speaking about two APIs of the ieee1394 framework.)

Which problems are solved by this patch? Do they outweigh the problems
it creates? The latter have been discussed. Dismissing them as Other
People's Problems does not nullify them.

Where is the agreed-upon, published plan for removal of features in
ieee1394?
--
Stefan Richter
-=====-=-=-= -=-- =-=--
http://arcgraph.de/sr/

2005-04-20 16:35:37

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, 2005-04-20 at 18:31 +0200, Stefan Richter wrote:
> Arjan van de Ven wrote:
> > On Wed, 2005-04-20 at 00:00 +0200, Stefan Richter wrote:
> >>There are users (though not in "the" kernel at the moment)
> > nor for the last 5 months... how long will it be ?
>
> Have there been problems with the API during the past 5 months, except
> that several kernel trees are using some parts of the API? (We are
> actually speaking about two APIs of the ieee1394 framework.)
>
> Which problems are solved by this patch?

exports and the functions beneath them cause the kernel binary be bigger
than needed. If nothing is using an api.. is it really the right one?


2005-04-20 17:10:09

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Arjan van de Ven wrote:
> If nothing is using an api

Check the archive.
--
Stefan Richter
-=====-=-=-= -=-- =-=--
http://arcgraph.de/sr/

2005-04-20 19:19:49

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, 2005-04-20 at 19:09 +0200, Stefan Richter wrote:
> Arjan van de Ven wrote:
> > If nothing is using an api
>
> Check the archive.

I don't care and in fact ignore external drivers that don't ever want to
get upstream. If there is a driver that wants this surely it wants to go
upstream soonish ?


2005-04-20 20:34:43

by Dan Dennedy

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, 2005-04-20 at 18:31 +0200, Stefan Richter wrote:
> Arjan van de Ven wrote:
> > On Wed, 2005-04-20 at 00:00 +0200, Stefan Richter wrote:
> >>There are users (though not in "the" kernel at the moment)
> > nor for the last 5 months... how long will it be ?
>
> Have there been problems with the API during the past 5 months, except
> that several kernel trees are using some parts of the API? (We are
> actually speaking about two APIs of the ieee1394 framework.)
>
> Which problems are solved by this patch? Do they outweigh the problems
> it creates? The latter have been discussed. Dismissing them as Other
> People's Problems does not nullify them.
>
> Where is the agreed-upon, published plan for removal of features in
> ieee1394?

Of course, there is none. In an unofficial capacity, there is just the
opinion and votes of the subsystem maintainers, extended team (e.g.,
Stefan and me), and kernel maintainers (Linus, Andrew, and Marcello).
The users are represented by the subsystem maintainers and extended
team. So, Arjan's opinion does not matter except to offer some voice of
reason.

I think I was first to raise objection upon the initial submission some
months back, and then I just observed the dialog. There are technical
merits for removal of the external symbols that I accept. I also accept
that we have no way of maintaining any sort of stable subsystem for
external projects we are not aware of or who are not communicating with
us about their requirements (it goes beyond just a stable interface).

Based upon my experience of several years on this project there is only
one external kernel module project we need to consider because that
developer has been involved and voiced requirements. That is Arne
Caspari's (The Imaging Source) DFG/1394 driver.

I vote to remove external symbols not used by the Linux1394.org modules
or the module at http://sourceforge.net/projects/video-2-1394/
Of course, I may be voted down, but I ask the others to be realistic
about what we are arguing for (i.e. just being defensive?) and consider
the fact that there are valid reasons for their removal.


2005-04-20 20:49:38

by Adrian Bunk

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, Apr 20, 2005 at 04:32:24PM -0400, Dan Dennedy wrote:
>...
> Based upon my experience of several years on this project there is only
> one external kernel module project we need to consider because that
> developer has been involved and voiced requirements. That is Arne
> Caspari's (The Imaging Source) DFG/1394 driver.

The ideal solution would be to get this driver included in the kernel.
Are there any reasons against including it?

> I vote to remove external symbols not used by the Linux1394.org modules
> or the module at http://sourceforge.net/projects/video-2-1394/
> Of course, I may be voted down, but I ask the others to be realistic
> about what we are arguing for (i.e. just being defensive?) and consider
> the fact that there are valid reasons for their removal.

The DFG/1394 driver requires hpsb_read and hpsb_write.

Are there any Linux1394.org modules that are not in 2.6.12-rc2-mm3?

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

2005-04-20 23:43:30

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Adrian Bunk wrote:
> On Wed, Apr 20, 2005 at 04:32:24PM -0400, Dan Dennedy wrote:
>>Based upon my experience of several years on this project there is only
>>one external kernel module project we need to consider because that
>>developer has been involved and voiced requirements. That is Arne
>>Caspari's (The Imaging Source) DFG/1394 driver.
>
> The ideal solution would be to get this driver included in the kernel.
> Are there any reasons against including it?

Arne stated his reasons: http://marc.theaimsgroup.com/?m=110361846312128
(BTW, does Arne have commit access to the linux1394 repo yet?)

>>I vote to remove external symbols not used by the Linux1394.org modules
>>or the module at http://sourceforge.net/projects/video-2-1394/
>>Of course, I may be voted down, but I ask the others to be realistic
>>about what we are arguing for (i.e. just being defensive?) and consider
>>the fact that there are valid reasons for their removal.

Yes, there are. (Although one of the reasons -- size of the kernel -- is
in effect neglectible.) So let's see if further opinions are coming in
at the lists now. Then let's decide how to manage removal of the stuff.

> The DFG/1394 driver requires hpsb_read and hpsb_write.
>
> Are there any Linux1394.org modules that are not in 2.6.12-rc2-mm3?

No.
--
Stefan Richter
-=====-=-=-= -=-- =-=-=
http://arcgraph.de/sr/

2005-04-27 21:12:36

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Dan Dennedy wrote on 2005-04-20:
> There are technical
> merits for removal of the external symbols that I accept. I also accept
> that we have no way of maintaining any sort of stable subsystem for
> external projects we are not aware of or who are not communicating with
> us about their requirements (it goes beyond just a stable interface).
...
> I vote to remove external symbols not used by the Linux1394.org modules
> or the module at http://sourceforge.net/projects/video-2-1394/

Nobody else posted specific requirements so far. So let's clean up the
API. How about this:
- Determine a date or event at which unused symbols and functions will
vanish. ("Unused": Not used by the mainline drivers and video-2-1394
or any other project of similar scope of which the linux1394
maintainers are informed soon enough.)
- Add an according entry to Documentation/feature-removal-schedule.txt.
- Add warning comments next to obsolete EXPORT_SYMBOLs. Add warning
printks to obsolete functions? (If there are any.)
Are there proposals for a date? How about end of June?
--
Stefan Richter
-=====-=-=-= -=-- ==-==
http://arcgraph.de/sr/

2005-04-27 21:35:05

by Jesper Juhl

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, 27 Apr 2005, Stefan Richter wrote:

> Dan Dennedy wrote on 2005-04-20:
> > There are technical
> > merits for removal of the external symbols that I accept. I also accept
> > that we have no way of maintaining any sort of stable subsystem for
> > external projects we are not aware of or who are not communicating with
> > us about their requirements (it goes beyond just a stable interface).
> ...
> > I vote to remove external symbols not used by the Linux1394.org modules
> > or the module at http://sourceforge.net/projects/video-2-1394/
>
> Nobody else posted specific requirements so far. So let's clean up the API.
> How about this:
<snip>
> - Add warning comments next to obsolete EXPORT_SYMBOLs. Add warning
> printks to obsolete functions? (If there are any.)

how about just adding __deprecated to them?

--
Jesper Juhl

2005-04-27 21:46:25

by Adrian Bunk

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, Apr 27, 2005 at 11:38:11PM +0200, Jesper Juhl wrote:
> On Wed, 27 Apr 2005, Stefan Richter wrote:
>
> > Dan Dennedy wrote on 2005-04-20:
> > > There are technical
> > > merits for removal of the external symbols that I accept. I also accept
> > > that we have no way of maintaining any sort of stable subsystem for
> > > external projects we are not aware of or who are not communicating with
> > > us about their requirements (it goes beyond just a stable interface).
> > ...
> > > I vote to remove external symbols not used by the Linux1394.org modules
> > > or the module at http://sourceforge.net/projects/video-2-1394/
> >
> > Nobody else posted specific requirements so far. So let's clean up the API.
> > How about this:
> <snip>
> > - Add warning comments next to obsolete EXPORT_SYMBOLs. Add warning
> > printks to obsolete functions? (If there are any.)
>
> how about just adding __deprecated to them?

Some of these functions might have valid users (e.g. from the same
file).

__deprecated_for_modules (already in -mm) is the right solution.

> Jesper Juhl

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

2005-05-12 22:30:00

by Adrian Bunk

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, Apr 27, 2005 at 11:44:08PM +0200, Adrian Bunk wrote:
> On Wed, Apr 27, 2005 at 11:38:11PM +0200, Jesper Juhl wrote:
> > On Wed, 27 Apr 2005, Stefan Richter wrote:
> >
> > > Dan Dennedy wrote on 2005-04-20:
> > > > There are technical
> > > > merits for removal of the external symbols that I accept. I also accept
> > > > that we have no way of maintaining any sort of stable subsystem for
> > > > external projects we are not aware of or who are not communicating with
> > > > us about their requirements (it goes beyond just a stable interface).
> > > ...
> > > > I vote to remove external symbols not used by the Linux1394.org modules
> > > > or the module at http://sourceforge.net/projects/video-2-1394/
> > >
> > > Nobody else posted specific requirements so far. So let's clean up the API.
> > > How about this:
> > <snip>
> > > - Add warning comments next to obsolete EXPORT_SYMBOLs. Add warning
> > > printks to obsolete functions? (If there are any.)
> >
> > how about just adding __deprecated to them?
>
> Some of these functions might have valid users (e.g. from the same
> file).
>
> __deprecated_for_modules (already in -mm) is the right solution.

Correcting myself:

Unfortunately, this doesn't work since this also gives you a bunch of
warnings for usage that doesn't use the EXPORT_SYMBOL's (e.g. if the
whole ieee1394 stuff is built as a module).

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

2005-05-12 22:33:16

by Adrian Bunk

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

On Wed, Apr 27, 2005 at 11:11:59PM +0200, Stefan Richter wrote:
> Dan Dennedy wrote on 2005-04-20:
> >There are technical
> >merits for removal of the external symbols that I accept. I also accept
> >that we have no way of maintaining any sort of stable subsystem for
> >external projects we are not aware of or who are not communicating with
> >us about their requirements (it goes beyond just a stable interface).
> ...
> >I vote to remove external symbols not used by the Linux1394.org modules
> >or the module at http://sourceforge.net/projects/video-2-1394/
>
> Nobody else posted specific requirements so far. So let's clean up the
> API. How about this:
> - Determine a date or event at which unused symbols and functions will
> vanish. ("Unused": Not used by the mainline drivers and video-2-1394
> or any other project of similar scope of which the linux1394
> maintainers are informed soon enough.)

OK.

> - Add an according entry to Documentation/feature-removal-schedule.txt.

OK.

> - Add warning comments next to obsolete EXPORT_SYMBOLs. Add warning
> printks to obsolete functions? (If there are any.)

OK.

> Are there proposals for a date? How about end of June?

OK (now it's perhaps beginning of Juli).


Is this OK for everyone?
If yes, I can prepare a patch.


> Stefan Richter

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

2005-05-12 23:29:15

by Stefan Richter

[permalink] [raw]
Subject: Re: [2.6 patch] drivers/ieee1394/: remove unneeded EXPORT_SYMBOL's

Adrian Bunk wrote:
> Is this OK for everyone?
> If yes, I can prepare a patch.

I think it is finally time to do so. I even wanted to prepare a patch
but got distracted...
--
Stefan Richter
-=====-=-=-= -=-= -==-=
http://arcgraph.de/sr/

2005-05-13 14:01:29

by Adrian Bunk

[permalink] [raw]
Subject: [2.6 patch] drivers/ieee1394/: schedule unused EXPORT_SYMBOL's for removal

On Fri, May 13, 2005 at 01:24:32AM +0200, Stefan Richter wrote:
> Adrian Bunk wrote:
> >Is this OK for everyone?
> >If yes, I can prepare a patch.
>
> I think it is finally time to do so. I even wanted to prepare a patch
> but got distracted...

Patch below.

> Stefan Richter

cu
Adrian


<-- snip -->


This patch schedules unused EXPORT_SYMBOL's for removal.

Signed-off-by: Adrian Bunk <[email protected]>

---

Documentation/feature-removal-schedule.txt | 21 ++++++++++++++
drivers/ieee1394/ieee1394_core.c | 31 +++++++++++++++++++++
2 files changed, 52 insertions(+)

--- linux-2.6.12-rc4-mm1-full/Documentation/feature-removal-schedule.txt.old 2005-05-13 15:19:54.000000000 +0200
+++ linux-2.6.12-rc4-mm1-full/Documentation/feature-removal-schedule.txt 2005-05-13 15:29:24.000000000 +0200
@@ -93,0 +94,21 @@
+
+---------------------------
+
+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
+When: July 2005
+Files: drivers/ieee1394/ieee1394_core.c
+Why: No modular usage in the kernel.
+Who: Adrian Bunk <[email protected]>
--- linux-2.6.12-rc4-mm1-full/drivers/ieee1394/ieee1394_core.c.old 2005-05-13 15:19:34.000000000 +0200
+++ linux-2.6.12-rc4-mm1-full/drivers/ieee1394/ieee1394_core.c 2005-05-13 15:28:17.000000000 +0200
@@ -1226,7 +1226,13 @@
EXPORT_SYMBOL(hpsb_alloc_packet);
EXPORT_SYMBOL(hpsb_free_packet);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(hpsb_send_phy_config);
+
EXPORT_SYMBOL(hpsb_send_packet);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(hpsb_send_packet_and_wait);
+
EXPORT_SYMBOL(hpsb_reset_bus);
EXPORT_SYMBOL(hpsb_bus_reset);
@@ -1265,6 +1271,11 @@
EXPORT_SYMBOL(hpsb_get_hostinfo_bykey);
EXPORT_SYMBOL(hpsb_set_hostinfo);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(highlevel_add_host);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(highlevel_remove_host);
+
EXPORT_SYMBOL(highlevel_host_reset);

@@ -1275,4 +1286,6 @@
EXPORT_SYMBOL(hpsb_unregister_protocol);
EXPORT_SYMBOL(ieee1394_bus_type);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(nodemgr_for_each_host);

@@ -1312,18 +1325,36 @@

/** csr1212.c **/
+
+/* EXPORT_SYMBOLs scheduled for removal */
EXPORT_SYMBOL(csr1212_create_csr);
EXPORT_SYMBOL(csr1212_init_local_csr);
EXPORT_SYMBOL(csr1212_new_immediate);
+
EXPORT_SYMBOL(csr1212_new_directory);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(csr1212_associate_keyval);
+
EXPORT_SYMBOL(csr1212_attach_keyval_to_directory);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(csr1212_new_string_descriptor_leaf);
+
EXPORT_SYMBOL(csr1212_detach_keyval_from_directory);
EXPORT_SYMBOL(csr1212_release_keyval);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(csr1212_destroy_csr);
+
EXPORT_SYMBOL(csr1212_read);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(csr1212_generate_csr_image);
+
EXPORT_SYMBOL(csr1212_parse_keyval);
+
+/* EXPORT_SYMBOL scheduled for removal */
EXPORT_SYMBOL(csr1212_parse_csr);
+
EXPORT_SYMBOL(_csr1212_read_keyval);
EXPORT_SYMBOL(_csr1212_destroy_keyval);