diff options
author | Mark Hatle <mark.hatle@amd.com> | 2023-01-05 14:38:13 -0800 |
---|---|---|
committer | Mark Hatle <mark.hatle@amd.com> | 2023-04-10 09:03:22 -0700 |
commit | 6b245dfaa86e44f18aff552f78e7ea1041e68cb9 (patch) | |
tree | a5d242787176b1fed0e3a87fc2abf761dfcea49b | |
parent | b077838b2febffdb63ddc7fc0d70ada438be0257 (diff) | |
download | meta-xilinx-6b245dfaa86e44f18aff552f78e7ea1041e68cb9.tar.gz |
xilinx-deprecated: Remove support for BOARD and BOARD VARIANT
We still warn the user. This will likely be moved to fatal in the near
future, but this would impact some automated test cases so preserve it as
a warning for now.
Also remove all of the support infrastructure for the BOARD/BOARD_VARIANT
values.
Signed-off-by: Mark Hatle <mark.hatle@amd.com>
7 files changed, 1 insertions, 116 deletions
diff --git a/meta-xilinx-core/classes/xilinx-deprecated.bbclass b/meta-xilinx-core/classes/xilinx-deprecated.bbclass index b9a44fc7..2e372a9b 100644 --- a/meta-xilinx-core/classes/xilinx-deprecated.bbclass +++ b/meta-xilinx-core/classes/xilinx-deprecated.bbclass | |||
@@ -7,62 +7,7 @@ xilinx_deprecated_config_eventhandler[eventmask] = "bb.event.ConfigParsed" | |||
7 | python xilinx_deprecated_config_eventhandler () { | 7 | python xilinx_deprecated_config_eventhandler () { |
8 | # Check for BOARD & BOARD_VARIANT usage | 8 | # Check for BOARD & BOARD_VARIANT usage |
9 | if d.getVar('BOARD') or d.getVar('BOARD_VARIANT'): | 9 | if d.getVar('BOARD') or d.getVar('BOARD_VARIANT'): |
10 | bb.warn("Deprecated BOARD (%s) or BOARD_VARIANT (%s) is being used." % (d.getVar('BOARD'), d.getVar('BOARD_VARIANT'))) | 10 | bb.warn("Deprecated BOARD (%s) or BOARD_VARIANT (%s) is being used, they are no longer supported and are ignored." % (d.getVar('BOARD'), d.getVar('BOARD_VARIANT'))) |
11 | |||
12 | if d.getVar('MACHINE') != d.getVar('ORIG_MACHINE'): | ||
13 | if d.getVar('BOARD_VARIANT') or d.getVar('BOARD') == d.getVar('MACHINE'): | ||
14 | if not check_conf_exists("conf/machine/${MACHINE}.conf", d): | ||
15 | mach_path = os.path.join(d.getVar('TOPDIR'), "conf/machine", d.getVar('MACHINE') + '.conf') | ||
16 | bb.utils.mkdirhier(os.path.dirname(mach_path)) | ||
17 | bb.warn('Generating (board_variant) MACHINE file: %s' % mach_path) | ||
18 | with open(mach_path, "w") as f: | ||
19 | f.write('#@TYPE: Machine\n') | ||
20 | f.write('#@NAME: %s\n' % d.getVar('MACHINE')) | ||
21 | f.write('#@DESCRIPTION: Generated %s machine\n' % d.getVar('MACHINE')) | ||
22 | f.write('\n') | ||
23 | f.write('#### Preamble\n') | ||
24 | f.write('''MACHINEOVERRIDES =. "${@['', '%s:']['%s' != '${MACHINE}']}"\n''' % (d.getVar('MACHINE'), d.getVar('MACHINE'))) | ||
25 | f.write('#### Regular settings follow\n') | ||
26 | f.write('\n') | ||
27 | f.write('unset BOARD\n') | ||
28 | f.write('unset BOARD_VARIANT\n') | ||
29 | f.write('\n') | ||
30 | f.write('DEFAULTTUNE ?= "%s"\n' % d.getVar('DEFAULTTUNE')) | ||
31 | if d.getVar('TUNE_FEATURES:tune-microblaze'): | ||
32 | f.write('TUNE_FEATURES:tune-microblaze ?= "%s"\n' % d.getVar('TUNE_FEATURES')) | ||
33 | if d.getVar('SOC_VARIANT'): | ||
34 | f.write('SOC_VARIANT ?= "%s"\n' % d.getVar('SOC_VARIANT')) | ||
35 | f.write('\n') | ||
36 | f.write('require conf/machine/%s.conf\n' % [d.getVar('ORIG_MACHINE'), d.getVar('BOARD')][bool(d.getVar('BOARD_VARIANT'))]) | ||
37 | f.write('\n') | ||
38 | f.write('#### No additional settings should be after the Postamble\n') | ||
39 | f.write('#### Postamble\n') | ||
40 | f.write('''PACKAGE_EXTRA_ARCHS:append = "${@['', ' %s']['%s' != "${MACHINE}"]}"\n''' % ((d.getVar('MACHINE_ARCH'), d.getVar('MACHINE')))) | ||
41 | bb.warn('Note: The generated machine conf file may be incomplete. If so copy the missing settings from the original conf files.') | ||
42 | bb.warn('In the future use: MACHINE = "%s"' % d.getVar('MACHINE')) | ||
43 | if d.getVar('BOARD') and d.getVar('BOARD') != d.getVar('MACHINE'): | ||
44 | if not check_conf_exists("conf/machine/${BOARD}.conf", d): | ||
45 | mach_path = os.path.join(d.getVar('TOPDIR'), "conf/machine", d.getVar('BOARD') + '.conf') | ||
46 | bb.utils.mkdirhier(os.path.dirname(mach_path)) | ||
47 | bb.warn('Generating (board) MACHINE file: %s' % mach_path) | ||
48 | with open(mach_path, "w") as f: | ||
49 | f.write('#@TYPE: Machine\n') | ||
50 | f.write('#@NAME: %s\n' % d.getVar('BOARD')) | ||
51 | f.write('#@DESCRIPTION: Generated %s machine\n' % d.getVar('BOARD')) | ||
52 | f.write('\n') | ||
53 | f.write('#### Preamble\n') | ||
54 | f.write('''MACHINEOVERRIDES =. "${@['', '%s:']['%s' != '${MACHINE}']}"\n''' % (d.getVar('BOARD'), d.getVar('BOARD'))) | ||
55 | f.write('#### Regular settings follow\n') | ||
56 | f.write('\n') | ||
57 | f.write('unset BOARD\n') | ||
58 | f.write('unset BOARD_VARIANT\n') | ||
59 | f.write('require conf/machine/%s.conf\n' % d.getVar('ORIG_MACHINE')) | ||
60 | f.write('\n') | ||
61 | f.write('#### No additional settings should be after the Postamble\n') | ||
62 | f.write('#### Postamble\n') | ||
63 | f.write('''PACKAGE_EXTRA_ARCHS:append = "${@['', ' %s']['%s' != "${MACHINE}"]}"\n''' % ((d.getVar('BOARD_ARCH'), d.getVar('BOARD')))) | ||
64 | bb.warn('Note: The generated machine conf file may be incomplete. If so copy the missing settings from the original conf files.') | ||
65 | d.delVar('ORIG_MACHINE') | ||
66 | 11 | ||
67 | msg_list = d.getVarFlags('XILINX_DEPRECATED') or [] | 12 | msg_list = d.getVarFlags('XILINX_DEPRECATED') or [] |
68 | for msg_source in msg_list: | 13 | for msg_source in msg_list: |
diff --git a/meta-xilinx-core/conf/machine/include/xilinx-board-post.inc b/meta-xilinx-core/conf/machine/include/xilinx-board-post.inc deleted file mode 100644 index 6f05eddd..00000000 --- a/meta-xilinx-core/conf/machine/include/xilinx-board-post.inc +++ /dev/null | |||
@@ -1,6 +0,0 @@ | |||
1 | BOARD_ARCH ?= "${@['${BOARD}', '${MACHINE_ARCH}'][d.getVar('BOARD')=='']}" | ||
2 | BOARDVARIANT_ARCH ?= "${@['${BOARD}_${BOARD_VARIANT}','${BOARD_ARCH}'][d.getVar('BOARD_VARIANT')=='']}" | ||
3 | |||
4 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' ${BOARD_ARCH}']['${BOARD}' != '' and '${BOARD_ARCH}' != '${MACHINE_ARCH}']}" | ||
5 | # We don't add BOARDVARIANT_ARCH as that will be automatic via MACHINE | ||
6 | #PACKAGE_EXTRA_ARCHS:append = "${@['', ' ${BOARDVARIANT_ARCH}']['${BOARD_VARIANT}' != '' and '${BOARDVARIANT_ARCH}' != '${MACHINE_ARCH}']}" | ||
diff --git a/meta-xilinx-core/conf/machine/include/xilinx-board-pre.inc b/meta-xilinx-core/conf/machine/include/xilinx-board-pre.inc deleted file mode 100644 index 8fea5ce2..00000000 --- a/meta-xilinx-core/conf/machine/include/xilinx-board-pre.inc +++ /dev/null | |||
@@ -1,14 +0,0 @@ | |||
1 | BOARD ??= "" | ||
2 | BOARD_VARIANT ??= "" | ||
3 | |||
4 | # Change the MACHINE to the BOARD-VARIANT (or BOARD) | ||
5 | ORIG_MACHINE := "${MACHINE}" | ||
6 | MACHINE := "${@['${BOARD}-${BOARD_VARIANT}', '${BOARD}' or '${ORIG_MACHINE}']['${BOARD_VARIANT}' == '']}" | ||
7 | |||
8 | # Add the BOARD and BOARD_VARIANT to the overrides | ||
9 | MACHINEOVERRIDES =. "${@['', '${BOARD}:']['${BOARD}' != '' and '${MACHINE}' != '${BOARD}']}" | ||
10 | # We don't add BOARD-BOARD_VARIANT as that will be automatic via MACHINE | ||
11 | #MACHINEOVERRIDES =. "${@['', '${BOARD}-${BOARD_VARIANT}:']['${BOARD_VARIANT}' != '' and '${MACHINE}' != '${BOARD_VARIANT}']}" | ||
12 | |||
13 | include conf/machine/include/${BOARD}.inc | ||
14 | include conf/machine/include/${BOARD}-${BOARD_VARIANT}.inc | ||
diff --git a/meta-xilinx-core/conf/machine/microblaze-generic.conf b/meta-xilinx-core/conf/machine/microblaze-generic.conf index eeade810..5a6bd084 100644 --- a/meta-xilinx-core/conf/machine/microblaze-generic.conf +++ b/meta-xilinx-core/conf/machine/microblaze-generic.conf | |||
@@ -2,11 +2,6 @@ | |||
2 | #@NAME: microblaze-generic | 2 | #@NAME: microblaze-generic |
3 | #@DESCRIPTION: Machine configuration for the microblaze-generic devices | 3 | #@DESCRIPTION: Machine configuration for the microblaze-generic devices |
4 | 4 | ||
5 | # Deprecated board config | ||
6 | USE_BOARD = "${@"conf/machine/include/xilinx-board-pre.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
7 | require ${USE_BOARD} | ||
8 | unset USE_BOARD | ||
9 | |||
10 | #### Preamble | 5 | #### Preamble |
11 | MACHINEOVERRIDES =. "${@['', 'microblaze-generic:']['microblaze-generic' != '${MACHINE}']}" | 6 | MACHINEOVERRIDES =. "${@['', 'microblaze-generic:']['microblaze-generic' != '${MACHINE}']}" |
12 | #### Regular settings follow | 7 | #### Regular settings follow |
@@ -97,8 +92,3 @@ QB_NETWORK_DEVICE = "-net nic,netdev=net0,macaddr=@MAC@" | |||
97 | #### No additional settings should be after the Postamble | 92 | #### No additional settings should be after the Postamble |
98 | #### Postamble | 93 | #### Postamble |
99 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' ${MB_MACHINE_ARCH}']['microblaze-generic' != "${MACHINE}"]}" | 94 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' ${MB_MACHINE_ARCH}']['microblaze-generic' != "${MACHINE}"]}" |
100 | |||
101 | # Deprecated board config | ||
102 | USE_BOARD = "${@"conf/machine/include/xilinx-board-post.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
103 | require ${USE_BOARD} | ||
104 | unset USE_BOARD | ||
diff --git a/meta-xilinx-core/conf/machine/versal-generic.conf b/meta-xilinx-core/conf/machine/versal-generic.conf index 7811073b..eb06426f 100644 --- a/meta-xilinx-core/conf/machine/versal-generic.conf +++ b/meta-xilinx-core/conf/machine/versal-generic.conf | |||
@@ -2,11 +2,6 @@ | |||
2 | #@NAME: versal-generic | 2 | #@NAME: versal-generic |
3 | #@DESCRIPTION: Machine configuration for the versal-generic devices | 3 | #@DESCRIPTION: Machine configuration for the versal-generic devices |
4 | 4 | ||
5 | # Deprecated board config | ||
6 | USE_BOARD = "${@"conf/machine/include/xilinx-board-pre.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
7 | require ${USE_BOARD} | ||
8 | unset USE_BOARD | ||
9 | |||
10 | #### Preamble | 5 | #### Preamble |
11 | MACHINEOVERRIDES =. "${@['', 'versal-generic:']['versal-generic' != '${MACHINE}']}" | 6 | MACHINEOVERRIDES =. "${@['', 'versal-generic:']['versal-generic' != '${MACHINE}']}" |
12 | #### Regular settings follow | 7 | #### Regular settings follow |
@@ -120,8 +115,3 @@ QB_OPT_APPEND += " -plm-args '${QB_PLM_OPT}'" | |||
120 | #### No additional settings should be after the Postamble | 115 | #### No additional settings should be after the Postamble |
121 | #### Postamble | 116 | #### Postamble |
122 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' versal_generic']['versal-generic' != "${MACHINE}"]}" | 117 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' versal_generic']['versal-generic' != "${MACHINE}"]}" |
123 | |||
124 | # Deprecated board config | ||
125 | USE_BOARD = "${@"conf/machine/include/xilinx-board-post.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
126 | require ${USE_BOARD} | ||
127 | unset USE_BOARD | ||
diff --git a/meta-xilinx-core/conf/machine/zynq-generic.conf b/meta-xilinx-core/conf/machine/zynq-generic.conf index a4e6e725..df82a683 100644 --- a/meta-xilinx-core/conf/machine/zynq-generic.conf +++ b/meta-xilinx-core/conf/machine/zynq-generic.conf | |||
@@ -2,11 +2,6 @@ | |||
2 | #@NAME: zynq-generic | 2 | #@NAME: zynq-generic |
3 | #@DESCRIPTION: Machine configuration for the zynq-generic devices | 3 | #@DESCRIPTION: Machine configuration for the zynq-generic devices |
4 | 4 | ||
5 | # Deprecated board config | ||
6 | USE_BOARD = "${@"conf/machine/include/xilinx-board-pre.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
7 | require ${USE_BOARD} | ||
8 | unset USE_BOARD | ||
9 | |||
10 | #### Preamble | 5 | #### Preamble |
11 | MACHINEOVERRIDES =. "${@['', 'zynq-generic:']['zynq-generic' != '${MACHINE}']}" | 6 | MACHINEOVERRIDES =. "${@['', 'zynq-generic:']['zynq-generic' != '${MACHINE}']}" |
12 | #### Regular settings follow | 7 | #### Regular settings follow |
@@ -96,8 +91,3 @@ QB_OPT_APPEND = " \ | |||
96 | #### No additional settings should be after the Postamble | 91 | #### No additional settings should be after the Postamble |
97 | #### Postamble | 92 | #### Postamble |
98 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' zynq_generic']['zynq-generic' != "${MACHINE}"]}" | 93 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' zynq_generic']['zynq-generic' != "${MACHINE}"]}" |
99 | |||
100 | # Deprecated board config | ||
101 | USE_BOARD = "${@"conf/machine/include/xilinx-board-post.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
102 | require ${USE_BOARD} | ||
103 | unset USE_BOARD | ||
diff --git a/meta-xilinx-core/conf/machine/zynqmp-generic.conf b/meta-xilinx-core/conf/machine/zynqmp-generic.conf index bf95c117..e7b47003 100644 --- a/meta-xilinx-core/conf/machine/zynqmp-generic.conf +++ b/meta-xilinx-core/conf/machine/zynqmp-generic.conf | |||
@@ -2,11 +2,6 @@ | |||
2 | #@NAME: zynqmp-generic | 2 | #@NAME: zynqmp-generic |
3 | #@DESCRIPTION: Machine configuration for the zynqmp-generic devices | 3 | #@DESCRIPTION: Machine configuration for the zynqmp-generic devices |
4 | 4 | ||
5 | # Deprecated board config | ||
6 | USE_BOARD = "${@"conf/machine/include/xilinx-board-pre.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
7 | require ${USE_BOARD} | ||
8 | unset USE_BOARD | ||
9 | |||
10 | #### Preamble | 5 | #### Preamble |
11 | MACHINEOVERRIDES =. "${@['', 'zynqmp-generic:']['zynqmp-generic' != '${MACHINE}']}" | 6 | MACHINEOVERRIDES =. "${@['', 'zynqmp-generic:']['zynqmp-generic' != '${MACHINE}']}" |
12 | #### Regular settings follow | 7 | #### Regular settings follow |
@@ -129,8 +124,3 @@ do_write_qemuboot_conf[depends] += "u-boot-xlnx-uenv:do_deploy" | |||
129 | #### No additional settings should be after the Postamble | 124 | #### No additional settings should be after the Postamble |
130 | #### Postamble | 125 | #### Postamble |
131 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' zynqmp_generic']['zynqmp-generic' != "${MACHINE}"]}" | 126 | PACKAGE_EXTRA_ARCHS:append = "${@['', ' zynqmp_generic']['zynqmp-generic' != "${MACHINE}"]}" |
132 | |||
133 | # Deprecated board config | ||
134 | USE_BOARD = "${@"conf/machine/include/xilinx-board-post.inc" if d.getVar("BOARD") or d.getVar("BOARD_VARIANT") else ""}" | ||
135 | require ${USE_BOARD} | ||
136 | unset USE_BOARD | ||