From d44f7d1e4ffba1848e90198d0bfbb239fda094ca Mon Sep 17 00:00:00 2001 From: AndreiGrozav Date: Thu, 18 Jan 2018 14:46:08 +0200 Subject: [PATCH] axi_ad9434: Fix bad parameter definition Assigning the value of a local parameter(localparam) to a parameter will end up with a conflict(not highlighted by the tool). In this case, the parameter type was defined as a string instead of an integer. Furthermore, this scenario leads to an undesired choice between primitive types. --- library/axi_ad9434/axi_ad9434.v | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/library/axi_ad9434/axi_ad9434.v b/library/axi_ad9434/axi_ad9434.v index a97733d1d..1ab3bd5f6 100644 --- a/library/axi_ad9434/axi_ad9434.v +++ b/library/axi_ad9434/axi_ad9434.v @@ -38,7 +38,7 @@ module axi_ad9434 #( parameter ID = 0, - parameter DEVICE_TYPE = SERIES7, + parameter DEVICE_TYPE = 0, parameter IO_DELAY_GROUP = "dev_if_delay_group") ( // physical interface @@ -82,9 +82,6 @@ module axi_ad9434 #( input [ 2:0] s_axi_awprot, input [ 2:0] s_axi_arprot); - localparam SERIES7 = 0; - localparam SERIES6 = 1; - // internal clocks & resets wire adc_rst;