Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extend mcu gen pads #493

Merged
merged 6 commits into from
Apr 30, 2024
Merged

Conversation

davidmallasen
Copy link
Member

@davidmallasen davidmallasen commented Apr 23, 2024

Extend the pad configuration so it includes an optional mapping to the side of the die. An example of this extra configuration is as follows:

module pad_cell_inout #(
    parameter int unsigned PADATTR = 16,
    parameter int unsigned SIDE = 0, // 0: top, 1: right, 2: bottom, 3: left
) (
    input logic pad_in_i,
    input logic pad_oe_i,
    output logic pad_out_o,
    inout logic pad_io,
    input logic [PADATTR-1:0] pad_attributes_i
);
    if (SIDE == 0) begin
        // Instantiate the top mapping pad cell
    end else if (SIDE == 1) begin
        // Instantiate the right mapping pad cell
    end else if
    // ...
endmodule

@davideschiavone
Copy link
Member

On top of adding the enum for orientarion in the script, can you also add it in the core_v_mini_mcu_pkg so that the pad design can rely on such SV enum rather than hardwiring 1 2 3 4?

@davidmallasen
Copy link
Member Author

Done @davideschiavone . Should I modify the ones in hw/simulation/ or in hw/fpga/? I see the ones for simulation have an extra parameter that the ones for fpga don't have

@davideschiavone
Copy link
Member

I would say both for completeness

@davideschiavone davideschiavone self-requested a review April 25, 2024 13:30
@davideschiavone davideschiavone changed the title Extend mcu gen pads extend mcu gen pads Apr 29, 2024
@davideschiavone davideschiavone merged commit a639054 into esl-epfl:main Apr 30, 2024
4 checks passed
@davidmallasen davidmallasen deleted the extend-mcu-gen-pads branch April 30, 2024 07:49
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.

2 participants