sanitycheck: cleanup hardware map display
Add a static method for dumping hardware map and reuse it across the script reducing duplicated code. Fixes #21475 Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This commit is contained in:
parent
9abc0e2efd
commit
0c69c285ec
1 changed files with 21 additions and 21 deletions
|
@ -3850,13 +3850,7 @@ class HardwareMap:
|
||||||
hwm = hwm + new
|
hwm = hwm + new
|
||||||
|
|
||||||
logger.info("Registered devices:")
|
logger.info("Registered devices:")
|
||||||
print("")
|
self.dump(hwm)
|
||||||
table = []
|
|
||||||
header = ["Platform", "ID", "Serial device"]
|
|
||||||
for p in sorted(hwm, key=lambda i: i['platform']):
|
|
||||||
platform = p.get('platform')
|
|
||||||
table.append([platform, p.get('id', None), p.get('serial')])
|
|
||||||
print(tabulate(table, headers=header, tablefmt="github"))
|
|
||||||
|
|
||||||
with open(hwm_file, 'w') as yaml_file:
|
with open(hwm_file, 'w') as yaml_file:
|
||||||
yaml.dump(hwm, yaml_file, default_flow_style=False)
|
yaml.dump(hwm, yaml_file, default_flow_style=False)
|
||||||
|
@ -3865,7 +3859,25 @@ class HardwareMap:
|
||||||
# create new file
|
# create new file
|
||||||
with open(hwm_file, 'w') as yaml_file:
|
with open(hwm_file, 'w') as yaml_file:
|
||||||
yaml.dump(self.detected, yaml_file, default_flow_style=False)
|
yaml.dump(self.detected, yaml_file, default_flow_style=False)
|
||||||
|
logger.info("Detected devices:")
|
||||||
|
self.dump(self.detected)
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
def dump(hwmap=[], filtered=[], header=[], connected_only=False):
|
||||||
|
print("")
|
||||||
|
table = []
|
||||||
|
if not header:
|
||||||
|
header = ["Platform", "ID", "Serial device"]
|
||||||
|
for p in sorted(hwmap, key=lambda i: i['platform']):
|
||||||
|
platform = p.get('platform')
|
||||||
|
connected = p.get('connected', False)
|
||||||
|
if filtered and platform not in filtered:
|
||||||
|
continue
|
||||||
|
|
||||||
|
if not connected_only or connected:
|
||||||
|
table.append([platform, p.get('id', None), p.get('serial')])
|
||||||
|
|
||||||
|
print(tabulate(table, headers=header, tablefmt="github"))
|
||||||
|
|
||||||
options = None
|
options = None
|
||||||
|
|
||||||
|
@ -3932,14 +3944,8 @@ def main():
|
||||||
hwm.load_hardware_map(options.hardware_map)
|
hwm.load_hardware_map(options.hardware_map)
|
||||||
|
|
||||||
logger.info("Available devices:")
|
logger.info("Available devices:")
|
||||||
print("")
|
|
||||||
table = []
|
table = []
|
||||||
header = ["Platform", "ID", "Serial device"]
|
hwm.dump(hwmap=hwm.connected_hardware, connected_only=True)
|
||||||
for p in hwm.connected_hardware:
|
|
||||||
platform = p.get('platform')
|
|
||||||
if p['connected']:
|
|
||||||
table.append([platform, p.get('id', None), p['serial']])
|
|
||||||
print(tabulate(table, headers=header, tablefmt="github"))
|
|
||||||
return
|
return
|
||||||
|
|
||||||
if options.west_runner and not options.west_flash:
|
if options.west_runner and not options.west_flash:
|
||||||
|
@ -4198,13 +4204,7 @@ def main():
|
||||||
|
|
||||||
if options.device_testing:
|
if options.device_testing:
|
||||||
print("\nDevice testing on:")
|
print("\nDevice testing on:")
|
||||||
table = []
|
hwm.dump(suite.connected_hardware, suite.selected_platforms)
|
||||||
header = ["Platform", "ID", "Serial device"]
|
|
||||||
for p in suite.connected_hardware:
|
|
||||||
platform = p.get('platform')
|
|
||||||
if p['connected'] and platform in suite.selected_platforms:
|
|
||||||
table.append([platform, p.get('id', None), p['serial']])
|
|
||||||
print(tabulate(table, headers=header, tablefmt="github"))
|
|
||||||
print("")
|
print("")
|
||||||
|
|
||||||
if options.dry_run:
|
if options.dry_run:
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue