search

chipsalliance / f4pga

FOSS Flow For FPGA
https://f4pga.org
Apache License 2.0
358 stars 48 forks source link

Remove --platform option and export part name to `part_name`. #565

Closed kboronski-ant closed 2 years ago

kboronski-ant commented 2 years ago

Platform option is too ambiguous. Part name should be required to be specified explicitely, then exposed as part_name variable so a correct pinmap could be used.

See https://github.com/chipsalliance/f4pga/blob/9b140e9d345c943e954eddcd49cf27ca569e9a2e/f4pga/platforms/xc7a50t.json#L101

It will always default to xc7a35tcsg324-1, unless the user explicitly sets part_name to something else. This makes --platform completely unnecessary.

tmichalak commented 2 years ago

Fixed with #568