Re: [rtc-linux] Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Friday 23 February 2007 4:09 am, Alessandro Zummo wrote: > > given that Greg also agreed, you have my > Acked-By: Alessandro Zummo <[EMAIL PROTECTED]> Thanks. I'll re-issue this series in a bit with those signoffs, after after the oops fix. Then I expect it should sit in MM for a while. - Dave - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [rtc-linux] Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thu, 22 Feb 2007 14:21:36 -0800 David Brownell <[EMAIL PROTECTED]> wrote: > > > > thanks for your work. The interface system was originally > > in place because it seemed the right-thing-to-do. if it isn't, > > better to remove it ;) > > The Right Thing may also have changed a bit over time! eheh :) given that Greg also agreed, you have my Acked-By: Alessandro Zummo <[EMAIL PROTECTED]> -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [rtc-linux] Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thu, 22 Feb 2007 14:21:36 -0800 David Brownell [EMAIL PROTECTED] wrote: thanks for your work. The interface system was originally in place because it seemed the right-thing-to-do. if it isn't, better to remove it ;) The Right Thing may also have changed a bit over time! eheh :) given that Greg also agreed, you have my Acked-By: Alessandro Zummo [EMAIL PROTECTED] -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [rtc-linux] Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Friday 23 February 2007 4:09 am, Alessandro Zummo wrote: given that Greg also agreed, you have my Acked-By: Alessandro Zummo [EMAIL PROTECTED] Thanks. I'll re-issue this series in a bit with those signoffs, after after the oops fix. Then I expect it should sit in MM for a while. - Dave - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thursday 22 February 2007 1:37 pm, Alessandro Zummo wro > Hi David, > > thanks for your work. The interface system was originally > in place because it seemed the right-thing-to-do. if it isn't, > better to remove it ;) The Right Thing may also have changed a bit over time! - Dave - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thu, 22 Feb 2007 12:44:31 -0800 David Brownell <[EMAIL PROTECTED]> wrote: > > All of these looks good to me, feel free to add an: > > Acked-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> > > to them if you want. > > Thanks ... > > > > thanks for doing the class_device removal work, I appreciate it. > > Actually the class_device removal work was because that was the > only way to clean the RTC suspend()/remove() patch out of my > queue, where it's been sitting since not log after those class > suspend/resume methods started to circulate, early last summer. > > It'll be good to have that finally gone! Hi David, thanks for your work. The interface system was originally in place because it seemed the right-thing-to-do. if it isn't, better to remove it ;) -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thursday 22 February 2007 10:09 am, Greg KH wrote: > On Wed, Feb 21, 2007 at 07:45:14PM -0800, David Brownell wrote: > > > > The main point of this series of patches is that last one, which I'll > > circulate just a bit more widely. (Although I think the first four are > > also nice cleanups!) It might be the first example of a framework that > > uses the "new" class level suspend()/resume() calls to offload drivers. > > > > .. > > All of these looks good to me, feel free to add an: > Acked-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> > to them if you want. Thanks ... > thanks for doing the class_device removal work, I appreciate it. Actually the class_device removal work was because that was the only way to clean the RTC suspend()/remove() patch out of my queue, where it's been sitting since not log after those class suspend/resume methods started to circulate, early last summer. It'll be good to have that finally gone! - Dave - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Wed, Feb 21, 2007 at 07:45:14PM -0800, David Brownell wrote: > Hi, > > Following this message (on the RTC list) are six patches: > > - Remove the /sys/class/rtc-dev class_device, and a class_interface > - Use "struct rtc_device" in the external interface, not class_device > - Simplify the sysfs attribute handling, removing a class_interface > - Simplify the /proc/driver/rtc handling, removing the last class_interface > - Remove the class_device in "struct rtc_device"; now suspend()/resume() work > - Implement class suspend()/resume() so the system clock is updated on resume > > The main point of this series of patches is that last one, which I'll > circulate just a bit more widely. (Although I think the first four are > also nice cleanups!) It might be the first example of a framework that > uses the "new" class level suspend()/resume() calls to offload drivers. > > Other than actually using that new PM infrastructure, this series should > be interesting since it addresses one of the few remaining obstacles to > having the "Generic Time-Of-Day" (GTOD) framework be fully generic, in > the sense of working with whatever RTC is available on the platform (which > includes ones accessed through I2C or SPI, so that spinlocked access is > insufficient) and removing arch-specific RTC hooks. > > This has been lightly tested on one of the ARMs that doesn't yet have > new-style dynamic tick working. I'm sure fault paths need tweaking yet. All of these looks good to me, feel free to add an: Acked-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> to them if you want. thanks for doing the class_device removal work, I appreciate it. greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Wed, Feb 21, 2007 at 07:45:14PM -0800, David Brownell wrote: Hi, Following this message (on the RTC list) are six patches: - Remove the /sys/class/rtc-dev class_device, and a class_interface - Use struct rtc_device in the external interface, not class_device - Simplify the sysfs attribute handling, removing a class_interface - Simplify the /proc/driver/rtc handling, removing the last class_interface - Remove the class_device in struct rtc_device; now suspend()/resume() work - Implement class suspend()/resume() so the system clock is updated on resume The main point of this series of patches is that last one, which I'll circulate just a bit more widely. (Although I think the first four are also nice cleanups!) It might be the first example of a framework that uses the new class level suspend()/resume() calls to offload drivers. Other than actually using that new PM infrastructure, this series should be interesting since it addresses one of the few remaining obstacles to having the Generic Time-Of-Day (GTOD) framework be fully generic, in the sense of working with whatever RTC is available on the platform (which includes ones accessed through I2C or SPI, so that spinlocked access is insufficient) and removing arch-specific RTC hooks. This has been lightly tested on one of the ARMs that doesn't yet have new-style dynamic tick working. I'm sure fault paths need tweaking yet. All of these looks good to me, feel free to add an: Acked-by: Greg Kroah-Hartman [EMAIL PROTECTED] to them if you want. thanks for doing the class_device removal work, I appreciate it. greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thursday 22 February 2007 10:09 am, Greg KH wrote: On Wed, Feb 21, 2007 at 07:45:14PM -0800, David Brownell wrote: The main point of this series of patches is that last one, which I'll circulate just a bit more widely. (Although I think the first four are also nice cleanups!) It might be the first example of a framework that uses the new class level suspend()/resume() calls to offload drivers. .. All of these looks good to me, feel free to add an: Acked-by: Greg Kroah-Hartman [EMAIL PROTECTED] to them if you want. Thanks ... thanks for doing the class_device removal work, I appreciate it. Actually the class_device removal work was because that was the only way to clean the RTC suspend()/remove() patch out of my queue, where it's been sitting since not log after those class suspend/resume methods started to circulate, early last summer. It'll be good to have that finally gone! - Dave - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thu, 22 Feb 2007 12:44:31 -0800 David Brownell [EMAIL PROTECTED] wrote: All of these looks good to me, feel free to add an: Acked-by: Greg Kroah-Hartman [EMAIL PROTECTED] to them if you want. Thanks ... thanks for doing the class_device removal work, I appreciate it. Actually the class_device removal work was because that was the only way to clean the RTC suspend()/remove() patch out of my queue, where it's been sitting since not log after those class suspend/resume methods started to circulate, early last summer. It'll be good to have that finally gone! Hi David, thanks for your work. The interface system was originally in place because it seemed the right-thing-to-do. if it isn't, better to remove it ;) -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
On Thursday 22 February 2007 1:37 pm, Alessandro Zummo wro Hi David, thanks for your work. The interface system was originally in place because it seemed the right-thing-to-do. if it isn't, better to remove it ;) The Right Thing may also have changed a bit over time! - Dave - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
Hi, Following this message (on the RTC list) are six patches: - Remove the /sys/class/rtc-dev class_device, and a class_interface - Use "struct rtc_device" in the external interface, not class_device - Simplify the sysfs attribute handling, removing a class_interface - Simplify the /proc/driver/rtc handling, removing the last class_interface - Remove the class_device in "struct rtc_device"; now suspend()/resume() work - Implement class suspend()/resume() so the system clock is updated on resume The main point of this series of patches is that last one, which I'll circulate just a bit more widely. (Although I think the first four are also nice cleanups!) It might be the first example of a framework that uses the "new" class level suspend()/resume() calls to offload drivers. Other than actually using that new PM infrastructure, this series should be interesting since it addresses one of the few remaining obstacles to having the "Generic Time-Of-Day" (GTOD) framework be fully generic, in the sense of working with whatever RTC is available on the platform (which includes ones accessed through I2C or SPI, so that spinlocked access is insufficient) and removing arch-specific RTC hooks. This has been lightly tested on one of the ARMs that doesn't yet have new-style dynamic tick working. I'm sure fault paths need tweaking yet. - Dave - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[patch/rfc 0/6] rtc framework: remove class_device, use class suspend()/resume()
Hi, Following this message (on the RTC list) are six patches: - Remove the /sys/class/rtc-dev class_device, and a class_interface - Use struct rtc_device in the external interface, not class_device - Simplify the sysfs attribute handling, removing a class_interface - Simplify the /proc/driver/rtc handling, removing the last class_interface - Remove the class_device in struct rtc_device; now suspend()/resume() work - Implement class suspend()/resume() so the system clock is updated on resume The main point of this series of patches is that last one, which I'll circulate just a bit more widely. (Although I think the first four are also nice cleanups!) It might be the first example of a framework that uses the new class level suspend()/resume() calls to offload drivers. Other than actually using that new PM infrastructure, this series should be interesting since it addresses one of the few remaining obstacles to having the Generic Time-Of-Day (GTOD) framework be fully generic, in the sense of working with whatever RTC is available on the platform (which includes ones accessed through I2C or SPI, so that spinlocked access is insufficient) and removing arch-specific RTC hooks. This has been lightly tested on one of the ARMs that doesn't yet have new-style dynamic tick working. I'm sure fault paths need tweaking yet. - Dave - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
