[rtc-linux] Re: [PATCH 1/3] rtc: rtc-twl: kill static variables

2016-11-23 Thread Nicolae Rosia
Hi, On Wed, Nov 23, 2016 at 10:46 AM, Alexandre Belloni wrote: >> Ok, could you please indicate which lines so I can fix them instead of >> wasting your time? >> > > checkpatch.pl --strict will find them for you Thanks, I had the wrong tab setting. -- You received this message because you are s

[rtc-linux] [PATCH v2 0/2] rtc: rtc-twl: cleanup

2016-11-23 Thread Nicolae Rosia
This is a cleanup and preparation of rtc-twl to reduce the usage of twl-core exported functions. The next patch series will further reduce the usage of exported functions but first it requires some changes in mfd/twl-core.c v2: - dropped patch "rtc: rtc-twl: use irq_of_parse_and_map helper" - fixe

[rtc-linux] [PATCH v2 1/2] rtc: rtc-twl: kill static variables

2016-11-23 Thread Nicolae Rosia
The current code uses static variables which prevent the use of multiple rtc twl instances. We also make it clear that this driver supports only TWL4030 and TWL6030 classes. Signed-off-by: Nicolae Rosia --- drivers/rtc/rtc-twl.c | 190 ++ 1 file ch

[rtc-linux] [PATCH v2 2/2] rtc: rtc-twl: make driver DT only

2016-11-23 Thread Nicolae Rosia
Since there are no platform based users and all users of this code are TI OMAP-based which is DT only, it makes sense to remove unused code. Signed-off-by: Nicolae Rosia --- drivers/rtc/Kconfig | 1 + drivers/rtc/rtc-twl.c | 12 +++- 2 files changed, 8 insertions(+), 5 deletions(-) d

[rtc-linux] Re: [PATCH 1/3] rtc: rtc-twl: kill static variables

2016-11-23 Thread Alexandre Belloni
On 23/11/2016 at 10:39:12 +0200, Nicolae Rosia wrote : > Hi, > Thanks for the review. > > On Wed, Nov 23, 2016 at 12:31 AM, Alexandre Belloni > wrote: > > Hi, > > > > A few alignments are off but I can fix that provided you answer the > > question below: > > > Ok, could you please indicate which

[rtc-linux] Re: [PATCH 2/3] rtc: rtc-twl: make driver DT only

2016-11-23 Thread Nicolae Rosia
Hi, On Wed, Nov 23, 2016 at 12:33 AM, Alexandre Belloni wrote: >> static struct platform_driver twl4030rtc_driver = { >> .probe = twl_rtc_probe, > > > You can then get rid of of_match_ptr() > Sure, doing it in v2. -- You received this message because you are subscribed to "rtc-l

[rtc-linux] Re: [PATCH 1/3] rtc: rtc-twl: kill static variables

2016-11-23 Thread Nicolae Rosia
Hi, Thanks for the review. On Wed, Nov 23, 2016 at 12:31 AM, Alexandre Belloni wrote: > Hi, > > A few alignments are off but I can fix that provided you answer the > question below: > Ok, could you please indicate which lines so I can fix them instead of wasting your time? > On 12/11/2016 at 12: