Fix cheshire_xilinx_top when USE_RESETN is undefined, wire sys_resetn is used but not declared.

[Aquaticfuller]

No description provided.

[paulsc96]

Hi,

I rebased this and cleaned it up for inclusion in main, but I am genuinely confused why you did this as it seems pointless.

When USE_RESETN is not defined, USE_RESET must be defined as things are currently implemented (admittedly a flawed design, but it's not my code).

However in this case, sys_resetn is declared and assigned to be ~sys_reset, so your change does not do anything.

Is there something I don't see here?

[Aquaticfuller]

Hi!
That is because I'm working on Flamingo, which doesn't use the cheshire_top_xilinx as its top level, instead it has another xilinx top module, and none of the macros are defined. But when the video processes that file, it returns an error about this. It might make more sense to directly remove this file from the Bender.yml of the Cheshire used by Flamingo.

[paulsc96]

Okay, that seems reasonable.