lkml.org 
[lkml]   [2005]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [2.6 patch] drivers/ide/: possible cleanups
From
Date
On Sun, 2005-05-01 at 16:29 +0200, Adrian Bunk wrote:
> On Sun, May 01, 2005 at 02:37:43PM +0100, Alan Cox wrote:
> > On Sad, 2005-04-30 at 21:07, Adrian Bunk wrote:
> > > This patch contains the following possible cleanups:
> > > - pci/cy82c693.c: make a needlessly global function static
> > > - remove the following unneeded EXPORT_SYMBOL's:
> > > - ide-taskfile.c: do_rw_taskfile
> > > - ide-iops.c: default_hwif_iops
> > > - ide-iops.c: default_hwif_transport
> > > - ide-iops.c: wait_for_ready
> >
> > default_*_ops are very much API items not currently used. You need them
> > if you
> > want to switch from mmio back to pio (eg doing S3 resume) although
> > nobody is currently doing that.
>
> My patch only removes the EXPORT_SYMBOL's.
>
> The functions themselves stay (since they are used), and if someone
> wants at some time in the future use them from a module, re-adding them
> will be trivial.

Hrm... well, that means if I ever want ide-pmac for example to be a
module, I'll have to add them back...

On the other hand, I agree that their names aren't very nice for
exported symbols... they should have been ide_*

Ben.


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

\
 
 \ /
  Last update: 2005-05-02 03:34    [W:0.032 / U:0.372 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site