From 4e9cc716642407a030c32354b58a30b9065722ca Mon Sep 17 00:00:00 2001
From: uvok
Date: Wed, 24 Dec 2025 19:36:32 +0100
Subject: Use localparam for divisor

---
 clkdiv.v | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/clkdiv.v b/clkdiv.v
index 7cf6cc3..5730d19 100644
--- a/clkdiv.v
+++ b/clkdiv.v
@@ -5,12 +5,16 @@ module clkdiv (
 );
 
 reg [23:0] counter;
+//  CLK is 27 MHz
+//  we want a 2Hz signal at the output
+//  and pin needs to *toggle twice* within one period
+//  27MHz / 4 = 6750000
+localparam DIVISOR = 24'd6_749_999;
 
 always @(posedge clk or negedge rst_i) begin
     if (!rst_i)
         counter <= 24'd0;
-//    else if (counter < 24'd1349_9999)       // 0.5s delay
-    else if (counter < 24'd674_9999)       // 0.5s delay
+    else if (counter < DIVISOR)
         counter <= counter + 1'b1;
     else
         counter <= 24'd0;
@@ -19,7 +23,7 @@ end
 always @(posedge clk or negedge rst_i) begin
     if (!rst_i)
         o_divclk <= 1'b0;
-    else if (counter == 24'd674_9999)       // 0.5s delay
+    else if (counter == DIVISOR)
         o_divclk <= ~o_divclk;
     else
         o_divclk <= o_divclk;
-- 
cgit v1.2.3