On Fri, Jul 25, 2014 at 3:57 PM, Chen-Yu Tsai <wens@xxxxxxxx> wrote:
> On Thu, Jul 24, 2014 at 12:02 AM, Varka Bhadram <varkabhadram@xxxxxxxxx> wrote:
>>
>> On Wednesday 23 July 2014 08:08 PM, Chen-Yu Tsai wrote:
>>>
>>> This patch introduces the driver for the RTC in the Allwinner A31 and
>>> A23 SoCs.
>>>
>>> Unlike the RTC found in A10/A20 SoCs, which was part of the timer, the
>>> RTC in A31/A23 are a separate hardware block, which also contain a few
>>> controls for the RTC block hardware (a regulator and RTC block GPIO pin
>>> latches), while also having separate interrupts for the alarms.
>>>
>>> The hardware is different enough to make a different driver for it.
>>>
>> (...)
>>
>>
>>> +Required properties:
>>> +- compatible : Should be "allwinner,sun6i-a31-rtc"
>>> +- reg: physical base address of the controller and length of memory
>>> mapped
>>> + region.
>>> +- interrupts: IRQ line for the RTC alarm 0.
>>> +
>>
>>
>> proper indentation..
>> - compatible : Should be "allwinner,sun6i-a31-rtc"
>>
>> - reg : physical base address of the controller and length of
>> memory mapped
>> region.
>>
>> - interrupts : IRQ line for the RTC alarm 0.
>> ....
>>
>>> +Example:
>>> +
>>
>>
>> (...)
>>
>>
>>> +
>>> + ret = devm_request_irq(&pdev->dev, chip->irq, sun6i_rtc_alarmirq,
>>> + 0, dev_name(&pdev->dev), chip);
>>
>>
>> should match open parenthesis...
>>
>>
>> devm_request_irq(&pdev->dev, chip->irq, sun6i_rtc_alarmirq,
>> 0, dev_name(&pdev->dev), chip);
>
> ok.
>
>>> + if (ret) {
>>> + dev_err(&pdev->dev, "Could not request IRQ\n");
>>> + return ret;
>>> + }
>>> +
>>> + /* clear the alarm counter value */
>>> + writel(0, chip->base + SUN6I_ALRM_COUNTER);
>>> +
>>> + /* disable counter alarm */
>>> + writel(0, chip->base + SUN6I_ALRM_EN);
>>> +
>>> + /* disable counter alarm interrupt */
>>> + writel(0, chip->base + SUN6I_ALRM_IRQ_EN);
>>> +
>>> + /* disable week alarm */
>>> + writel(0, chip->base + SUN6I_ALRM1_EN);
>>> +
>>> + /* disable week alarm interrupt */
>>> + writel(0, chip->base + SUN6I_ALRM1_IRQ_EN);
>>> +
>>> + /* clear counter alarm pending interrupts */
>>> + writel(SUN6I_ALRM_IRQ_STA_CNT_IRQ_PEND, chip->base +
>>> + SUN6I_ALRM_IRQ_STA);
>>> +
>>> + /* clear week alarm pending interrupts */
>>> + writel(SUN6I_ALRM1_IRQ_STA_WEEK_IRQ_PEND, chip->base +
>>> + SUN6I_ALRM1_IRQ_STA);
>>> +
>>> + /* disable alarm wakeup */
>>> + writel(0, chip->base + SUN6I_ALARM_CONFIG);
>>> +
>>> + chip->rtc = rtc_device_register("rtc-sun6i", &pdev->dev,
>>> + &sun6i_rtc_ops, THIS_MODULE);
>>
>>
>> dto....
>
> ok.
>
>>> + if (IS_ERR(chip->rtc)) {
>>> + dev_err(&pdev->dev, "unable to register device\n");
>>> + return PTR_ERR(chip->rtc);
>>> + }
>>> +
>>> + dev_info(&pdev->dev, "RTC enabled\n");
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static int sun6i_rtc_remove(struct platform_device *pdev)
>>> +{
>>> + struct sun6i_rtc_dev *chip = platform_get_drvdata(pdev);
>>> +
>>> + rtc_device_unregister(chip->rtc);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static const struct of_device_id sun6i_rtc_dt_ids[] = {
>>> + { .compatible = "allwinner,sun6i-a31-rtc" },
>>> + { /* sentinel */ },
>>> +};
>>> +MODULE_DEVICE_TABLE(of, sun6i_rtc_dt_ids);
>>> +
>>> +static struct platform_driver sun6i_rtc_driver = {
>>> + .probe = sun6i_rtc_probe,
>>> + .remove = sun6i_rtc_remove,
>>> + .driver = {
>>> + .name = "sun6i-rtc",
>>> + .owner = THIS_MODULE,
>>
>>
>> we can drop owner field....
>
> May I ask why this is not needed? Seems most if not all drivers
> set it.
>
> Thanks.
Never mind. I figured it out. Sorry for the noise.
>>> + .of_match_table = sun6i_rtc_dt_ids,
>>> + },
>>> +};
>>> +
>>> +module_platform_driver(sun6i_rtc_driver);
>>> +
>>> +MODULE_DESCRIPTION("sun6i RTC driver");
>>> +MODULE_AUTHOR("Chen-Yu Tsai <wens@xxxxxxxx>");
>>> +MODULE_LICENSE("GPL");
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
