parttool: remove exemption from style check
This commit is contained in:
parent
7458c1c1e2
commit
a3b581bd8c
1
.flake8
1
.flake8
|
@ -156,7 +156,6 @@ exclude =
|
||||||
components/espcoredump/test/test_espcoredump.py,
|
components/espcoredump/test/test_espcoredump.py,
|
||||||
components/nvs_flash/nvs_partition_generator/nvs_partition_gen.py,
|
components/nvs_flash/nvs_partition_generator/nvs_partition_gen.py,
|
||||||
components/partition_table/gen_esp32part.py,
|
components/partition_table/gen_esp32part.py,
|
||||||
components/partition_table/parttool.py,
|
|
||||||
components/partition_table/test_gen_esp32part_host/gen_esp32part_tests.py,
|
components/partition_table/test_gen_esp32part_host/gen_esp32part_tests.py,
|
||||||
components/protocomm/python/constants_pb2.py,
|
components/protocomm/python/constants_pb2.py,
|
||||||
components/protocomm/python/sec0_pb2.py,
|
components/protocomm/python/sec0_pb2.py,
|
||||||
|
|
|
@ -32,6 +32,7 @@ ESPTOOL_PY = os.path.join(IDF_COMPONENTS_PATH, "esptool_py", "esptool", "esptool
|
||||||
|
|
||||||
quiet = False
|
quiet = False
|
||||||
|
|
||||||
|
|
||||||
def status(msg):
|
def status(msg):
|
||||||
""" Print status message to stderr """
|
""" Print status message to stderr """
|
||||||
if not quiet:
|
if not quiet:
|
||||||
|
@ -92,7 +93,7 @@ def _get_partition(args):
|
||||||
elif args.partition_type and args.partition_subtype:
|
elif args.partition_type and args.partition_subtype:
|
||||||
partition = partition_table.find_by_type(args.partition_type, args.partition_subtype)
|
partition = partition_table.find_by_type(args.partition_type, args.partition_subtype)
|
||||||
elif args.partition_boot_default:
|
elif args.partition_boot_default:
|
||||||
search = [ "factory" ] + [ "ota_{}".format(d) for d in range(16) ]
|
search = ["factory"] + ["ota_{}".format(d) for d in range(16)]
|
||||||
for subtype in search:
|
for subtype in search:
|
||||||
partition = partition_table.find_by_type("app", subtype)
|
partition = partition_table.find_by_type("app", subtype)
|
||||||
if partition is not None:
|
if partition is not None:
|
||||||
|
@ -154,7 +155,7 @@ def get_partition_info(args):
|
||||||
partition = None
|
partition = None
|
||||||
|
|
||||||
if args.table:
|
if args.table:
|
||||||
partition_table = _get_partition_table(args)
|
partition_table = _get_partition_table(args)
|
||||||
|
|
||||||
if args.table.endswith(".csv"):
|
if args.table.endswith(".csv"):
|
||||||
partition_table = partition_table.to_csv()
|
partition_table = partition_table.to_csv()
|
||||||
|
@ -169,7 +170,7 @@ def get_partition_info(args):
|
||||||
|
|
||||||
if partition:
|
if partition:
|
||||||
info_dict = {
|
info_dict = {
|
||||||
"offset" : '0x{:x}'.format(partition.offset),
|
"offset": '0x{:x}'.format(partition.offset),
|
||||||
"size": '0x{:x}'.format(partition.size)
|
"size": '0x{:x}'.format(partition.size)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -181,7 +182,7 @@ def get_partition_info(args):
|
||||||
except KeyError:
|
except KeyError:
|
||||||
raise RuntimeError("Request for unknown partition info {}".format(info))
|
raise RuntimeError("Request for unknown partition info {}".format(info))
|
||||||
|
|
||||||
status("Requested partition information [{}]:".format( ", ".join(args.info)))
|
status("Requested partition information [{}]:".format(", ".join(args.info)))
|
||||||
print(" ".join(infos))
|
print(" ".join(infos))
|
||||||
else:
|
else:
|
||||||
status("Partition not found")
|
status("Partition not found")
|
||||||
|
@ -192,7 +193,7 @@ def generate_blank_partition_file(args):
|
||||||
stdout_binary = None
|
stdout_binary = None
|
||||||
|
|
||||||
partition = _get_and_check_partition(args)
|
partition = _get_and_check_partition(args)
|
||||||
output = b"\xFF" * partition.size
|
output = b"\xFF" * partition.size
|
||||||
|
|
||||||
try:
|
try:
|
||||||
stdout_binary = sys.stdout.buffer # Python 3
|
stdout_binary = sys.stdout.buffer # Python 3
|
||||||
|
@ -226,8 +227,8 @@ def main():
|
||||||
|
|
||||||
partition_selection_args.add_argument("--partition-name", "-n", help="name of the partition")
|
partition_selection_args.add_argument("--partition-name", "-n", help="name of the partition")
|
||||||
partition_selection_args.add_argument("--partition-type", "-t", help="type of the partition")
|
partition_selection_args.add_argument("--partition-type", "-t", help="type of the partition")
|
||||||
partition_selection_args.add_argument('--partition-boot-default', "-d", help='select the default boot partition, '+
|
partition_selection_args.add_argument('--partition-boot-default', "-d", help='select the default boot partition \
|
||||||
'using the same fallback logic as the IDF bootloader', action="store_true")
|
using the same fallback logic as the IDF bootloader', action="store_true")
|
||||||
|
|
||||||
parser.add_argument("--partition-subtype", "-s", help="subtype of the partition")
|
parser.add_argument("--partition-subtype", "-s", help="subtype of the partition")
|
||||||
|
|
||||||
|
@ -247,7 +248,8 @@ def main():
|
||||||
print_partition_info_subparser_info_type.add_argument("--info", help="type of partition information to get", nargs="+")
|
print_partition_info_subparser_info_type.add_argument("--info", help="type of partition information to get", nargs="+")
|
||||||
print_partition_info_subparser_info_type.add_argument("--table", help="dump the partition table to a file")
|
print_partition_info_subparser_info_type.add_argument("--table", help="dump the partition table to a file")
|
||||||
|
|
||||||
generate_blank_subparser = subparsers.add_parser("generate_blank_partition_file", help="generate a blank (all 0xFF) partition file of the specified partition that can be flashed to the device")
|
generate_blank_subparser = subparsers.add_parser("generate_blank_partition_file", help="generate a blank (all 0xFF) partition file of \
|
||||||
|
the specified partition that can be flashed to the device")
|
||||||
generate_blank_subparser.add_argument("--output", help="blank partition file filename")
|
generate_blank_subparser.add_argument("--output", help="blank partition file filename")
|
||||||
|
|
||||||
args = parser.parse_args()
|
args = parser.parse_args()
|
||||||
|
@ -274,4 +276,4 @@ def main():
|
||||||
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
main()
|
main()
|
||||||
|
|
Loading…
Reference in a new issue