On 9/23/21 2:42 AM, Shahab Vahedi wrote:
Hi Randy,
I have one minor comment.
On 9/23/21 5:18 AM, Randy Dunlap wrote:
--- linux-next-20210917.orig/include/soc/arc/timers.h
+++ linux-next-20210917/include/soc/arc/timers.h
@@ -17,7 +17,7 @@
#define ARC_REG_TIMER1_CNT 0x100 /* timer 1 count */
/* CTRL reg bits */
-#define TIMER_CTRL_IE (1 << 0) /* Interrupt when Count reaches limit
*/
+#define TIMER_CTRL_IEN (1 << 0) /* Interrupt when Count reaches limit
*/
Could you change the name to "ARC_TIMER_CTRL_IE" instead? and while at it,
please rename
"TIMER_CTRL_NH" to "ARC_TIMER_CTRL_NH" for obvious consistency reasons and
saving us
from future clashes. Thank you!
OK, no problem.
Thanks for the feedback.
--
~Randy
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
