summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJim Mussared <jim.mussared@gmail.com>2023-08-03 14:41:25 +1000
committerDamien George <damien@micropython.org>2023-11-03 13:58:07 +1100
commitc3c7c602da724b697a71c19ebce3e49c67e760f6 (patch)
tree81dca52b17d77824a380f193bd727ddc14edc0f4
parent2eda5138701d6a7d36f8d8e3700d136b3c1161b7 (diff)
rp2/boards/make-pins.py: Don't generate qstrs.
Also remove af-const header, as this is left over from the STM32 version and unused. This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>
-rw-r--r--ports/rp2/CMakeLists.txt5
-rwxr-xr-xports/rp2/boards/make-pins.py65
2 files changed, 2 insertions, 68 deletions
diff --git a/ports/rp2/CMakeLists.txt b/ports/rp2/CMakeLists.txt
index 16f2d8a24..4a0d2fbad 100644
--- a/ports/rp2/CMakeLists.txt
+++ b/ports/rp2/CMakeLists.txt
@@ -479,7 +479,6 @@ set(GEN_PINS_PREFIX "${MICROPY_BOARDS_DIR}/rp2_prefix.c")
set(GEN_PINS_MKPINS "${MICROPY_BOARDS_DIR}/make-pins.py")
set(GEN_PINS_SRC "${CMAKE_BINARY_DIR}/pins_${MICROPY_BOARD}.c")
set(GEN_PINS_HDR "${MICROPY_GENHDR_DIR}/pins.h")
-set(GEN_PINS_QSTR "${CMAKE_BINARY_DIR}/pins_qstr.h")
set(GEN_PINS_AF_CONST "${MICROPY_GENHDR_DIR}/pins_af_const.h")
set(GEN_PINS_AF_PY "${CMAKE_BINARY_DIR}/pins_af.py")
@@ -494,9 +493,9 @@ target_sources(${MICROPY_TARGET} PRIVATE
# Generate pins
add_custom_command(
- OUTPUT ${GEN_PINS_HDR} ${GEN_PINS_SRC} ${GEN_PINS_QSTR}
+ OUTPUT ${GEN_PINS_HDR} ${GEN_PINS_SRC}
COMMAND ${Python3_EXECUTABLE} ${GEN_PINS_MKPINS} ${GEN_PINS_CSV_ARG} --af ${GEN_PINS_AF_CSV} --prefix ${GEN_PINS_PREFIX}
- --hdr ${GEN_PINS_HDR} --qstr ${GEN_PINS_QSTR} --af-const ${GEN_PINS_AF_CONST} --af-py ${GEN_PINS_AF_PY} > ${GEN_PINS_SRC}
+ --hdr ${GEN_PINS_HDR} --af-py ${GEN_PINS_AF_PY} > ${GEN_PINS_SRC}
DEPENDS
${GEN_PINS_AF_CSV}
${GEN_PINS_BOARD_CSV}
diff --git a/ports/rp2/boards/make-pins.py b/ports/rp2/boards/make-pins.py
index 97c0ff638..4029d7b1c 100755
--- a/ports/rp2/boards/make-pins.py
+++ b/ports/rp2/boards/make-pins.py
@@ -68,15 +68,6 @@ class AlternateFunction(object):
def is_supported(self):
return self.supported
- def ptr(self):
- """Returns the numbered function (i.e. USART6) for this AF."""
- if self.fn_num is None:
- return self.func
- return "{:s}{:d}".format(self.func, self.fn_num)
-
- def mux_name(self):
- return "AF{:d}_{:s}".format(self.idx, self.ptr())
-
def print(self):
"""Prints the C representation of this AF."""
if self.supported:
@@ -88,9 +79,6 @@ class AlternateFunction(object):
fn_num = 0
print("({:d}, {:4s}, {:d}), // {:s}".format(self.idx, self.func, fn_num, self.af_str))
- def qstr_list(self):
- return [self.mux_name()]
-
class Pin(object):
"""Holds the information associated with a pin."""
@@ -164,13 +152,6 @@ class Pin(object):
if self.alt_fn_count > 0:
hdr_file.write("extern const machine_pin_af_obj_t pin_{:s}_af[];\n".format(n))
- def qstr_list(self):
- result = []
- for alt_fn in self.alt_fn:
- if alt_fn.is_supported():
- result += alt_fn.qstr_list()
- return result
-
class NamedPin(object):
def __init__(self, name, pin):
@@ -303,37 +284,6 @@ class Pins(object):
)
)
- def print_qstr(self, qstr_filename):
- with open(qstr_filename, "wt") as qstr_file:
- qstr_set = set([])
- for named_pin in self.cpu_pins:
- pin = named_pin.pin()
- if pin.is_board_pin():
- qstr_set |= set(pin.qstr_list())
- qstr_set |= set([named_pin.name()])
- for named_pin in self.board_pins:
- qstr_set |= set([named_pin.name()])
- for qstr in sorted(qstr_set):
- print("Q({})".format(qstr), file=qstr_file)
-
- def print_af_hdr(self, af_const_filename):
- with open(af_const_filename, "wt") as af_const_file:
- af_hdr_set = set([])
- mux_name_width = 0
- for named_pin in self.cpu_pins:
- pin = named_pin.pin()
- if pin.is_board_pin():
- for af in pin.alt_fn:
- if af.is_supported():
- mux_name = af.mux_name()
- af_hdr_set |= set([mux_name])
- if len(mux_name) > mux_name_width:
- mux_name_width = len(mux_name)
- for mux_name in sorted(af_hdr_set):
- key = "MP_OBJ_NEW_QSTR(MP_QSTR_{}),".format(mux_name)
- val = "MP_OBJ_NEW_SMALL_INT(GPIO_{})".format(mux_name)
- print(" { %-*s %s }," % (mux_name_width + 26, key, val), file=af_const_file)
-
def print_af_py(self, af_py_filename):
with open(af_py_filename, "wt") as af_py_file:
print("PINS_AF = (", file=af_py_file)
@@ -360,12 +310,6 @@ def main():
default="rp2_af.csv",
)
parser.add_argument(
- "--af-const",
- dest="af_const_filename",
- help="Specifies header file for alternate function constants.",
- default="build/pins_af_const.h",
- )
- parser.add_argument(
"--af-py",
dest="af_py_filename",
help="Specifies the filename for the python alternate function mappings.",
@@ -385,13 +329,6 @@ def main():
default="rp2_prefix.c",
)
parser.add_argument(
- "-q",
- "--qstr",
- dest="qstr_filename",
- help="Specifies name of generated qstr header file",
- default="build/pins_qstr.h",
- )
- parser.add_argument(
"-r",
"--hdr",
dest="hdr_filename",
@@ -419,8 +356,6 @@ def main():
print(prefix_file.read())
pins.print()
pins.print_header(args.hdr_filename, True)
- pins.print_qstr(args.qstr_filename)
- pins.print_af_hdr(args.af_const_filename)
pins.print_af_py(args.af_py_filename)