library/axi_ltc2387: Update GUI parameters

Signed-off-by: Stanca Pop <[email protected]>

Author: IuliaCMoldovan

library/axi_ltc2387/axi_ltc2387_channel.v

@@ -101,7 +101,13 @@ module axi_ltc2387_channel #(
 
   assign adc_pn_err_s = adc_pn_err;
 
-  // expected pattern
+  // expected patterns:
+  // 18-bit, one-lane: 10 1000 0001 1111 1100
+  // 16-bit, one-lane: 10 1000 0001 1111 11
+  // 18-bit, two-lane: 11 0011 0000 1111 1100
+  // 16-bit, one-lane: 11 0011 0000 1111 11
+  // basically, the 16-bit variant doesn't have the LSBs 00
+  // so we can sum this up as the 16-bit expected pattern + 2 zeroes for the 18-bit one
 
   generate
     if (TWOLANES == 1) begin

library/axi_ltc2387/axi_ltc2387_ip.tcl

@@ -31,49 +31,72 @@ adi_ip_files axi_ltc2387 [list \
 
 adi_ip_properties axi_ltc2387
 
-set cc [ipx::current_core]
-set page0 [ipgui::get_pagespec -name "Page 0" -component $cc]
+ipx::infer_bus_interface ref_clk xilinx.com:signal:clock_rtl:1.0 [ipx::current_core]
+ipx::infer_bus_interface dco_p xilinx.com:signal:clock_rtl:1.0 [ipx::current_core]
+ipx::infer_bus_interface dco_n xilinx.com:signal:clock_rtl:1.0 [ipx::current_core]
 
-ipx::infer_bus_interface ref_clk xilinx.com:signal:clock_rtl:1.0 $cc
-ipx::infer_bus_interface dco_p xilinx.com:signal:clock_rtl:1.0 $cc
-ipx::infer_bus_interface dco_n xilinx.com:signal:clock_rtl:1.0 $cc
+set cc [ipx::current_core]
 
-ipgui::add_static_text -name {Warning} -component $cc -parent $page0 -text {In one-lane mode, only 18-bit resolution is supported!}
+set page0 [ipgui::get_pagespec -name "Page 0" -component $cc]
 
 ipx::add_user_parameter ADC_RES $cc
 set_property value_resolve_type user [ipx::get_user_parameters ADC_RES -of_objects $cc]
 ipgui::add_param -name "ADC_RES" -component $cc -parent $page0
 set_property -dict [list \
-  "display_name" "ADC_RES" \
-  "layout" "horizontal" \
-  "tooltip" "ADC resolution" \
+  "display_name" "ADC resolution" \
+  "tooltip" "ADC_RES" \
   "widget" "radioGroup" \
+  "layout" "horizontal" \
 ] [ipgui::get_guiparamspec -name "ADC_RES" -component $cc]
 
 set_property -dict [list \
-  "value" "18" \
   "value_format" "long" \
   "value_validation_type" "list" \
-  "value_validation_list" "18 16" \
+  "value_validation_list" "16 18" \
 ] [ipx::get_user_parameters ADC_RES -of_objects $cc]
 
+ipx::add_user_parameter OUT_RES $cc
+set_property value_resolve_type user [ipx::get_user_parameters OUT_RES -of_objects $cc]
+ipgui::add_param -name "OUT_RES" -component $cc -parent $page0
+set_property -dict [list \
+  "display_name" "Output data width" \
+  "tooltip" "OUT_RES" \
+  "widget" "radioGroup" \
+  "layout" "horizontal" \
+] [ipgui::get_guiparamspec -name "OUT_RES" -component $cc]
+
+set_property -dict [list \
+  "value_format" "long" \
+  "value_validation_type" "list" \
+  "value_validation_list" "16 32" \
+] [ipx::get_user_parameters OUT_RES -of_objects $cc]
+
 ipx::add_user_parameter TWOLANES $cc
 set_property value_resolve_type user [ipx::get_user_parameters TWOLANES -of_objects $cc]
 ipgui::add_param -name "TWOLANES" -component $cc -parent $page0
 set_property -dict [list \
-  "display_name" "TWOLANES" \
-  "layout" "horizontal" \
-  "tooltip" "Two-lane mode (1) or one-lane mode (0)" \
+  "display_name" "Lane mode" \
+  "tooltip" "TWOLANES" \
   "widget" "radioGroup" \
+  "layout" "horizontal" \
 ] [ipgui::get_guiparamspec -name "TWOLANES" -component $cc]
 
 set_property -dict [list \
-  "value" "1" \
   "value_format" "long" \
   "value_validation_type" "list" \
-  "value_validation_list" "1 0" \
+  "value_validation_list" "0 1" \
 ] [ipx::get_user_parameters TWOLANES -of_objects $cc]
 
+# if TWOLANES=0, disable and tie to GND, ports db_p, db_n
+adi_set_ports_dependency "db_p" \
+  "(spirit:decode(id('MODELPARAM_VALUE.TWOLANES')) == 1)"
+adi_set_ports_dependency "db_n" \
+  "(spirit:decode(id('MODELPARAM_VALUE.TWOLANES')) == 1)"
+
+set_property driver_value 0 [ipx::get_ports -filter "direction==in" -of_objects $cc]
+
+#adi_add_auto_fpga_spec_params
+
 ipx::create_xgui_files $cc
 ipx::update_checksums $cc
 ipx::save_core $cc