The reg
type is only used for procedural assignments, but instance connections are treated more like continuous assignments.
Remove the reg
keyword from your outputs. Change:
output reg [2:0] s,
output reg [2:0] c);
to:
output [2:0] s,
output [2:0] c);