Skip to content

Conversation

zapta
Copy link

@zapta zapta commented Jul 9, 2025

This PR fixes the lint error reported in issue #352 by specifying explicitly the unused signals in the instantiation of SB_PLL40_CORE. The generated code was tested successfully with an actual Alhambra-ii FPGA board.

The new command generates this code (notice the added 'unused' section):

pll.v

/**
 * PLL configuration
 *
 * This Verilog module was generated automatically
 * using the icepll tool from the IceStorm project.
 * Use at your own risk.
 *
 * Given input frequency:        12.000 MHz
 * Requested output frequency:   48.000 MHz
 * Achieved output frequency:    48.000 MHz
 */

module pll(
	input  clock_in,
	output clock_out,
	output locked
	);

SB_PLL40_CORE #(
		.FEEDBACK_PATH("SIMPLE"),
		.DIVR(4'b0000),		// DIVR =  0
		.DIVF(7'b0111111),	// DIVF = 63
		.DIVQ(3'b100),		// DIVQ =  4
		.FILTER_RANGE(3'b001)	// FILTER_RANGE = 1
	) uut (
		.LOCK(locked),
		.RESETB(1'b1),
		.BYPASS(1'b0),
		.REFERENCECLK(clock_in),
		.PLLOUTCORE(clock_out),

		/* Unused */
		.PLLOUTGLOBAL(),
		.EXTFEEDBACK(),
		.LATCHINPUTVALUE(),
		.SDO(),
		.SDI(),
		.SCLK(),
		.DYNAMICDELAY()
		);

endmodule

…also the list of the unused PLL signals.
@zapta zapta changed the title [icepll] Fixing the lint error reported in issue #352 [icepll] Fixed the generated PLL lint errors from issue #352 Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant