Re: [patch] IDE driver model update

From: Pavel Machek (pavel@ucw.cz)
Date: Fri Oct 11 2002 - 18:06:32 EST


Hi!

> ChangeSet@1.696.19.2, 2002-10-07 10:27:16-07:00, mochel@osdl.org
> IDE: register ide driver for all ide drives; not just for disk drives.
>
> This adds
> struct device_driver gen_driver;
>
> to ide_driver_t, which is filled in with necessary fields when an ide
> driver calls ide_register_driver(). That then registers the driver with
> the driver model core.
>
> As a result, this gives us the following output in driverfs:
>
> # tree -d /sys/bus/ide/drivers/
> /sys/bus/ide/drivers/
> |-- ide-cdrom
> `-- ide-disk
>
> The suspend/resume callbacks in ide-disk.c have been temporarily
> disabled until the ide core implements generic methods which forward
> the calls to the drive drivers.
>
> diff -Nru a/drivers/ide/ide-disk.c b/drivers/ide/ide-disk.c
> --- a/drivers/ide/ide-disk.c Mon Oct 7 12:19:14 2002
> +++ b/drivers/ide/ide-disk.c Mon Oct 7 12:19:14 2002
> @@ -1550,14 +1550,6 @@
> /* This is just a hook for the overall driver tree.
> */
>
> -static struct device_driver idedisk_devdrv = {
> - .bus = &ide_bus_type,
> - .name = "IDE disk driver",
> -
> - .suspend = idedisk_suspend,
> - .resume = idedisk_resume,
> -};
> -

Here you killed the only reference to idedisk_suspend, yet
idedisk_suspend/idedisk_resume is needed if you don't want to eat
data.
                                                        Pavel

-- 
When do you have heart between your knees?
-
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/



This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:54 EST