Hi!

> +struct system_power_chip;
> +
> +struct system_power_ops {
> +     int (*restart)(struct system_power_chip *chip, enum reboot_mode mode,
> +                    char *cmd);
> +     int (*power_off_prepare)(struct system_power_chip *chip);
> +     int (*power_off)(struct system_power_chip *chip);
> +};
> +
> +struct system_power_chip {
> +     const struct system_power_ops *ops;
> +     struct list_head list;
> +     struct device *dev;
> +};

Is it useful to have two structures? AFAICT one would do.

Do we always have struct device * to work with? IMO we have nothing
suitable for example in the ACPI case. Would void * be more suitable?

Could you convert someting (acpi?) to the new framework as
demonstration?
                                                                Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Reply via email to