Re: [patch 07/12] Re: radio-sf16fmi cleanup

From: Domen Puncer
Date: Tue Mar 08 2005 - 05:30:05 EST


On 06/03/05 16:40 -0800, Andrew Morton wrote:
> domen@xxxxxxxxxxxx wrote:
> >
> >
> > This is small cleanup of radio-sf16fmi driver.
>
> Well, yes, but it is a functional change, no?
>
> Previously the kernel accepted the `sf16fm=' option. Now the users must
> switch over to, umm, `radio-sf16fmi.io=', yes?

Right.

>
>
>
> > Signed-off-by: Marcel Sebek <sebek64@xxxxxxx>
> > Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx>
> > ---
> >
> >
> > kj-domen/Documentation/kernel-parameters.txt | 3 ---
> > kj-domen/drivers/media/radio/radio-sf16fmi.c | 10 ----------
> > 2 files changed, 13 deletions(-)
> >
> > diff -puN Documentation/kernel-parameters.txt~kill_kernel_parameter-sf16fm Documentation/kernel-parameters.txt
> > --- kj/Documentation/kernel-parameters.txt~kill_kernel_parameter-sf16fm 2005-03-05 16:11:31.000000000 +0100
> > +++ kj-domen/Documentation/kernel-parameters.txt 2005-03-05 16:11:31.000000000 +0100
> > @@ -1148,9 +1148,6 @@ running once the system is up.
> >
> > serialnumber [BUGS=IA-32]
> >
> > - sf16fm= [HW] SF16FMI radio driver for Linux
> > - Format: <io>
> > -
> > sg_def_reserved_size=
> > [SCSI]
> >
> > diff -puN drivers/media/radio/radio-sf16fmi.c~kill_kernel_parameter-sf16fm drivers/media/radio/radio-sf16fmi.c
> > --- kj/drivers/media/radio/radio-sf16fmi.c~kill_kernel_parameter-sf16fm 2005-03-05 16:11:31.000000000 +0100
> > +++ kj-domen/drivers/media/radio/radio-sf16fmi.c 2005-03-05 16:11:31.000000000 +0100
> > @@ -326,13 +326,3 @@ static void __exit fmi_cleanup_module(vo
> >
> > module_init(fmi_init);
> > module_exit(fmi_cleanup_module);
> > -
> > -#ifndef MODULE
> > -static int __init fmi_setup_io(char *str)
> > -{
> > - get_option(&str, &io);
> > - return 1;
> > -}
> > -
> > -__setup("sf16fm=", fmi_setup_io);
> > -#endif
> > _
-
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/