timer: altera_timer: minor clean up

- Moved macro definitions to top
- Remove the penultimate comma in of_match ids

Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
Reviewed-by: Jagan Teki <jteki@openedev.com>

diff --git a/drivers/timer/altera_timer.c b/drivers/timer/altera_timer.c
index 288357c98bdafe99a39d1217fba34111b9a5a239..46a598ae9fcf01c95e8f6914630f773b5056321e 100644 (file)
--- a/drivers/timer/altera_timer.c
+++ b/drivers/timer/altera_timer.c
@@ -16,6 +16,11 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
+/* control register */
+#define ALTERA_TIMER_CONT      BIT(1)  /* Continuous mode */
+#define ALTERA_TIMER_START     BIT(2)  /* Start timer */
+#define ALTERA_TIMER_STOP      BIT(3)  /* Stop timer */
+
 struct altera_timer_regs {
        u32     status;         /* Timer status reg */
        u32     control;        /* Timer control reg */
@@ -30,11 +35,6 @@ struct altera_timer_platdata {
        unsigned long clock_rate;
 };
 
-/* control register */
-#define ALTERA_TIMER_CONT      BIT(1)  /* Continuous mode */
-#define ALTERA_TIMER_START     BIT(2)  /* Start timer */
-#define ALTERA_TIMER_STOP      BIT(3)  /* Stop timer */
-
 static int altera_timer_get_count(struct udevice *dev, unsigned long *count)
 {
        struct altera_timer_platdata *plat = dev->platdata;
@@ -88,8 +88,8 @@ static const struct timer_ops altera_timer_ops = {
 };
 
 static const struct udevice_id altera_timer_ids[] = {
-       { .compatible = "altr,timer-1.0", },
-       { }
+       { .compatible = "altr,timer-1.0" },
+       {}
 };
 
 U_BOOT_DRIVER(altera_timer) = {