Messages in this thread | | | Subject | Re: [PATCH 2/2] rtc: max77686: Use REGMAP_IRQ_REG for regmap-rtc-irqs initialisation | From | Krzysztof Kozlowski <> | Date | Wed, 02 Mar 2016 12:45:52 +0900 |
| |
On 02.03.2016 11:04, Laxman Dewangan wrote: > > On Wednesday 02 March 2016 06:30 AM, Krzysztof Kozlowski wrote: >> On 29.02.2016 21:58, Laxman Dewangan wrote: >>> use macro REGMAP_IRQ_REG from regmap.h to initialise the >>> regmap irq table for max77686. >> Why? Any benefits? >> >> > I think it is better coding to use macro for member initializations > instead of doing it locally and it improves readability. > > I got the similar comment from Lee's in my MFD patches to use macro for > such initialization and I agreed with him. > > As I was here on this driver, thought to use macro.
Sounds good, so this should be written in commit message.
BR, Krzysztof
| |