-----Original Message-----
From: Linus Walleij [mailto:linus.walleij@linaro.org]
Sent: Thursday, August 13, 2015 9:54 PM
To: Wang Dongsheng-B40534; John Stultz; Alessandro Zummo; Alexandre Belloni
Cc: Shawn Guo; Nair, Sandeep; Hans de Goede; Wang Huan-B18965; linux-arm-
kernel@lists.infradead.org; linux-kernel@vger.kernel.org; rtc-
linux@googlegroups.com
Subject: Re: [PATCH 2/2] soc/fsl: add ftm alarm driver for ls1021a platform

On Wed, Aug 12, 2015 at 7:53 AM, Dongsheng Wang
[off-list ref] wrote:

quoted

From: Wang Dongsheng <redacted>

Only Ftm0 can be used when system going to deep sleep. So this driver
to support ftm0 as a wakeup source.

Signed-off-by: Wang Dongsheng <redacted>
---
*V2*
Change Copyright 2014 to 2015.

(...)

quoted

+config FTM_ALARM
+       bool "FTM alarm driver"
+       depends on SOC_LS1021A
+       default n
+       help
+         Say y here to enable FTM alarm support.  The FTM alarm provides
+         alarm functions for wakeup system from deep sleep.  There is only
+         one FTM can be used in ALARM(FTM 0).

(...)

quoted

+static u32 time_to_cycle(unsigned long time)
+static u32 cycle_to_time(u32 cycle)
+static int ftm_set_alarm(u64 cycle)
+static irqreturn_t ftm_alarm_interrupt(int irq, void *dev_id)
+static ssize_t ftm_alarm_show(struct device *dev,
+                             struct device_attribute *attr,
+                             char *buf)
+static ssize_t ftm_alarm_store(struct device *dev,
+                              struct device_attribute *attr,
+                              const char *buf, size_t count)

(...)

quoted

+static struct device_attribute ftm_alarm_attributes = __ATTR(ftm_alarm, 0644,
+                       ftm_alarm_show, ftm_alarm_store);

If you're gonna invent ABIs, document then in Documentation/ABI/testing/*.

But I don't get it. Why is this driver not in drivers/rtc?

It does a subset of what an RTC does. The ioctl()'s of an RTC
can do what you want to do. And much much more.

If it can't do all an RTC can do, surely the RTC subsystem
can be augmented to host it anyway. It's way to close to
an RTC to have it's own random sysfs driver like this.

Unless I'm totally off, rewrite this to an RTC driver and post
it to the RTC maintainers.

FlexTimer is not a RTC device and not have any rtc deivce function. They belong to
different devices, why we need to register this to RTC framework? I am confused about this.

Now in freescale layerscape platform this driver is only for FlexTimer0, and not
fit for each flextimer. Because only FlexTimer0 still turn-on when system in the Deep Sleep.

If the "alarm" make you feel confused or mislead you think this is a RTC devices. I think
I need to change the "alarm" to "timer".

Regards,
-Dongsheng

-- 
-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.