1. 09 Mar, 2011 8 commits
    • Marcelo Roberto Jimenez's avatar
      RTC: Fix the cross interrupt issue on rtc-test. · a417493e
      Marcelo Roberto Jimenez authored
      The rtc-test driver is meant to provide a test/debug code for the RTC
      subsystem.
      
      The rtc-test driver simulates specific interrupts by echoing to the
      sys interface. Those were the update, alarm and periodic interrupts.
      
      As a side effect of the new implementation, any interrupt generated in
      the rtc-test driver would trigger the same code path in the generic
      code, and thus the distinction among interrupts gets lost.
      
      This patch preserves the previous behaviour of the rtc-test driver,
      where e.g. an update interrupt would not trigger an alarm or periodic
      interrupt, and vice-versa. In real world RTC drivers, this is not an
      issue, but in the rtc-test driver it may be interesting to distinguish
      these interrupts for testing purposes.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarMarcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      a417493e
    • Marcelo Roberto Jimenez's avatar
      RTC: Remove UIE and PIE information from the sa1100 driver proc. · 4cebe7aa
      Marcelo Roberto Jimenez authored
      This patch removes the UIE and PIE information that is now being
      supplied directly in the generic RTC code.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarMarcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      4cebe7aa
    • Marcelo Roberto Jimenez's avatar
      RTC: Include information about UIE and PIE in RTC driver proc. · bca8521c
      Marcelo Roberto Jimenez authored
      Generic RTC code is always able to provide the necessary information
      about update and periodic interrupts. This patch add such information to
      the proc interface.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarMarcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      bca8521c
    • John Stultz's avatar
      RTC: Clean out UIE icotl implementations · e428c6a2
      John Stultz authored
      With the generic RTC rework, the UIE mode irqs are handled
      in the generic layer, and only hardware specific ioctls
      get passed down to the rtc driver layer.
      
      So this patch removes the UIE mode ioctl handling in the rtc
      driver layer, which never get used.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      e428c6a2
    • John Stultz's avatar
      RTC: Cleanup rtc_class_ops->update_irq_enable() · 51ba60c5
      John Stultz authored
      Now that the generic code handles UIE mode irqs via periodic
      alarm interrupts, no one calls the
      rtc_class_ops->update_irq_enable() method anymore.
      
      This patch removes the driver hooks and implementations of
      update_irq_enable if no one else is calling it.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      51ba60c5
    • John Stultz's avatar
      RTC: Cleanup rtc_class_ops->irq_set_freq() · 696160fe
      John Stultz authored
      With the generic rtc code now emulating PIE mode irqs via an
      hrtimer, no one calls the rtc_class_ops->irq_set_freq call.
      
      This patch removes the hook and deletes the driver functions
      if no one else calls them.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      696160fe
    • John Stultz's avatar
      RTC: Cleanup rtc_class_ops->irq_set_state · 80d4bb51
      John Stultz authored
      With PIE mode interrupts now emulated in generic code via an hrtimer,
      no one calls rtc_class_ops->irq_set_state(), so this patch removes it
      along with driver implementations.
      
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: rtc-linux@googlegroups.com
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      80d4bb51
    • John Stultz's avatar
      RTC: Initialize kernel state from RTC · f44f7f96
      John Stultz authored
      Mark Brown pointed out a corner case: that RTC alarms should
      be allowed to be persistent across reboots if the hardware
      supported it.
      
      The rework of the generic layer to virtualize the RTC alarm
      virtualized much of the alarm handling, and removed the
      code used to read the alarm time from the hardware.
      
      Mark noted if we want the alarm to be persistent across
      reboots, we need to re-read the alarm value into the
      virtualized generic layer at boot up, so that the generic
      layer properly exposes that value.
      
      This patch restores much of the earlier removed
      rtc_read_alarm code and wires it in so that we
      set the kernel's alarm value to what we find in the
      hardware at boot time.
      
      NOTE: Not all hardware supports persistent RTC alarm state across
      system reset. rtc-cmos for example will keep the alarm time, but
      disables the AIE mode irq. Applications should not expect the RTC
      alarm to be valid after a system reset. We will preserve what
      we can, to represent the hardware state at boot, but its not
      guarenteed.
      
      Further, in the future, with multiplexed RTC alarms, the
      soonest alarm to fire may not be the one set via the /dev/rt
      ioctls. So an application may set the alarm with RTC_ALM_SET,
      but after a reset find that RTC_ALM_READ returns an earlier
      time. Again, we preserve what we can, but applications should
      not expect the RTC alarm state to persist across a system reset.
      
      Big thanks to Mark for pointing out the issue!
      Thanks also to Marcelo for helping think through the solution.
      
      CC: Mark Brown <broonie@opensource.wolfsonmicro.com>
      CC: Marcelo Roberto Jimenez <mroberto@cpti.cetuc.puc-rio.br>
      CC: Thomas Gleixner <tglx@linutronix.de>
      CC: Alessandro Zummo <a.zummo@towertech.it>
      CC: rtc-linux@googlegroups.com
      Reported-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
      Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
      f44f7f96
  2. 08 Mar, 2011 5 commits
  3. 07 Mar, 2011 9 commits
  4. 06 Mar, 2011 5 commits
  5. 05 Mar, 2011 13 commits