Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933400AbbHLBJQ (ORCPT ); Tue, 11 Aug 2015 21:09:16 -0400 Received: from mailout4.samsung.com ([203.254.224.34]:43831 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932883AbbHLBJO (ORCPT ); Tue, 11 Aug 2015 21:09:14 -0400 X-AuditID: cbfee68d-f79106d00000728c-94-55ca9cb81a6f Message-id: <55CA9CB9.1050007@samsung.com> Date: Wed, 12 Aug 2015 10:09:13 +0900 From: Joonyoung Shim User-Agent: Mozilla/5.0 (X11; Linux i686; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-version: 1.0 To: Krzysztof Kozlowski , rtc-linux@googlegroups.com Cc: linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, a.zummo@towertech.it, alexandre.belloni@free-electrons.com, cw00.choi@samsung.com Subject: Re: [PATCH 4/4] rtc: s3c: enable/disable clocks for alarm References: <1439292502-22912-1-git-send-email-jy0922.shim@samsung.com> <1439292502-22912-4-git-send-email-jy0922.shim@samsung.com> <55CA9330.9070700@samsung.com> In-reply-to: <55CA9330.9070700@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrIIsWRmVeSWpSXmKPExsWyRsSkUHfHnFOhBocPSVgsuXiV3aLj2mIm i+tfnrNavH5haHF51xw2ixnn9zFZ7O/sYHRg93iy6SKjx56JJ9k8+rasYvSYPu8nk8fnTXIB rFFcNimpOZllqUX6dglcGXun3mIu+Mhecff5aeYGxuVsXYycHBICJhK9m75C2WISF+6tB7K5 OIQEVjBKdN3sYe5i5AAr6r5mDRFfyigxYccuVgjnAaPElC19YN28AloSR7rPMIPYLAKqEtdu r2IEsdkE9CTubDvOBGKLCoRJnJnRwQJRLyjxY/I9MFtEwFfiwcqjLCBDmQVmMUpMffaUCWSz sICTxK/+Gohly4CWHW9gBWngFNCW2LLnMSNIDTPQgvsXtUDCzALyEpvXvGUGqZcQuMYusb5l NhvEQQIS3yYfYoH4RlZi0wFmiI8lJQ6uuMEygVFsFpKTZiFMnYVk6gJG5lWMoqkFyQXFSelF hnrFibnFpXnpesn5uZsYgTF3+t+z3h2Mtw9YH2IU4GBU4uE1mHgqVIg1say4MvcQoynQEROZ pUST84GRnVcSb2hsZmRhamJqbGRuaaYkzqso9TNYSCA9sSQ1OzW1ILUovqg0J7X4ECMTB6dU A2PhZOvHynzxe0/wzg1TPfHm38KpXYdLe0Ri3NRt9yvH9P/hU1m7/PDZd5whe579Ypv1/1bP jPOPrp/23MP59O/amuvsCspnWrU4eWoiQ/sWrfj077nztPDNubMeb7R5vu4Tm6LkpW+5U9av nhrz7P//VZkpdmdLvgYWXTJ9HPX5Ymjo5lP37e87KrEUZyQaajEXFScCAP/CkUK0AgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrHIsWRmVeSWpSXmKPExsVy+t9jAd0dc06FGpzfKWyx5OJVdouOa4uZ LK5/ec5q8fqFocXlXXPYLGac38dksb+zg9GB3ePJpouMHnsmnmTz6NuyitFj+ryfTB6fN8kF sEY1MNpkpCampBYppOYl56dk5qXbKnkHxzvHm5oZGOoaWlqYKynkJeam2iq5+AToumXmAJ2h pFCWmFMKFApILC5W0rfDNCE0xE3XAqYxQtc3JAiux8gADSSsYczYO/UWc8FH9oq7z08zNzAu Z+ti5OCQEDCR6L5m3cXICWSKSVy4tx4ozMUhJLCUUWLCjl2sEM4DRokpW/rYQKp4BbQkjnSf YQaxWQRUJa7dXsUIYrMJ6Enc2XacCcQWFQiTODOjgwWiXlDix+R7YLaIgK/Eg5VHWUCGMgvM YpSY+uwpE8gVwgJOEr/6ayCWLQNadryBFaSBU0BbYsuex4wgNcxAC+5f1AIJMwvIS2xe85Z5 AiPQEIQVsxCqZiGpWsDIvIpRIrUguaA4KT3XKC+1XK84Mbe4NC9dLzk/dxMjOK6fSe9gPLzL /RCjAAejEg+vwcRToUKsiWXFlbmHGCU4mJVEeG9YAYV4UxIrq1KL8uOLSnNSiw8xmgLDYCKz lGhyPjDl5JXEGxqbmBlZGpkbWhgZmyuJ8+qbbAoVEkhPLEnNTk0tSC2C6WPi4JRqYDQ7yTjT fcLtl3Wuay23N19/mJpi5uLpdfmsj4L139PuJbz3QoOvRlz4zc8qWr9k/834y9L/J7jvv695 Y4uuHltczu9pf3es1Au9ELh44h7Ha34/blzbqb77XdGz/4nf579xn7LrfD//g8lnCh2fL5Qw Y+/lvTE182HJWh3ziQmdes9lL05Iip2uxFKckWioxVxUnAgAkfPOKwEDAAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1223 Lines: 36 On 08/12/2015 09:28 AM, Krzysztof Kozlowski wrote: > On 11.08.2015 20:28, Joonyoung Shim wrote: >> The clock enable/disable codes for alarm have removed from > > What do you mean in this paragraph? The clock code was removing something? > >> 'commit 24e1455493da ("drivers/rtc/rtc-s3c.c: delete duplicate clock > > Remove the 'apostrophe. > >> control")' and the clocks keep disabling even if alarm is set, so alarm >> interrupt can't happen. > ...and the clocks are disabled even... > > >> >> The s3c_rtc_setaie function can be called several times with that >> enabled argument has same value, > ...several times with 'enabled' argument having same value > >> so it needs to check whether clocks is >> enabled or not. > s/is/are/ > >> >> Signed-off-by: Joonyoung Shim > > Please add Cc-stable and fixes tag. To backport the patch probably > you'll have to remove the dependency on previous patches. Thanks for the point, i didn't think it. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/