[Discuss OpenCPI] Zynq HDL Primitive Modification

From: Brian Padalino <bpadalino_at_gmail.com>
Date: Thu, 09 Jan 20 10:07:54 -0500

I figured out a missing piece of my ADALM-PLUTO puzzle. I was getting
placing errors about IO which couldn't be placed in an empty assembly. It
turned out that the zynq_ps instantiation was using some generics that
didn't work for the device, specifically:

  - C_PACKAGE_NAME
  - C_DQ_WIDTH
  - C_DQS_WIDTH
  - C_DM_WIDTH

I added the following generics to the zynq_ps entity:

  - DQ_WIDTH : positive := 32
  - PACKAGE_NAME : string := "clg484"

I then made the following changes to the instantiation:

  C_DQ_WIDTH => DQ_WIDTH,
  C_DQS_WIDTH => DQ_WIDTH/8,
  C_DM_WIDTH => DQ_WIDTH/8,
  C_PACKAGE_NAME => PACKAGE_NAME

I don't know if this is the best way to do it, but it seemed to work for me
to allow for the old stuff to still work while giving me the ability to
over ride it. Best practice might be to make the PACKAGE_NAME into an
enumerated type since a string is too generic especially with no checking.

So, to the real question, what is the preferred method for contributing
some of these changes back to the community? And, are there issues seen
with what I did here? Does it follow your best practices or should things
be modified?

Thanks,
Brian


Received on Thu Jan 09 2020 - 15:07:54 CST