Skip to content

Commit 2ba8c7d

Browse files
jones-drewpalmer-dabbelt
authored andcommitted
riscv: Don't duplicate __ALTERNATIVE_CFG in __ALTERNATIVE_CFG_2
Build __ALTERNATIVE_CFG_2 by adding on to __ALTERNATIVE_CFG rather than duplicating it. Signed-off-by: Andrew Jones <[email protected]> Tested-by: Lad Prabhakar <[email protected]> Acked-by: Conor Dooley <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Palmer Dabbelt <[email protected]>
1 parent 9abf231 commit 2ba8c7d

File tree

1 file changed

+2
-16
lines changed

1 file changed

+2
-16
lines changed

arch/riscv/include/asm/alternative-macros.h

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -49,14 +49,7 @@
4949

5050
.macro __ALTERNATIVE_CFG_2 old_c, new_c_1, vendor_id_1, errata_id_1, enable_1, \
5151
new_c_2, vendor_id_2, errata_id_2, enable_2
52-
886 :
53-
.option push
54-
.option norvc
55-
.option norelax
56-
\old_c
57-
.option pop
58-
887 :
59-
ALT_NEW_CONTENT \vendor_id_1, \errata_id_1, \enable_1, \new_c_1
52+
__ALTERNATIVE_CFG \old_c, \new_c_1, \vendor_id_1, \errata_id_1, \enable_1
6053
ALT_NEW_CONTENT \vendor_id_2, \errata_id_2, \enable_2, \new_c_2
6154
.endm
6255

@@ -116,14 +109,7 @@
116109
enable_1, \
117110
new_c_2, vendor_id_2, errata_id_2, \
118111
enable_2) \
119-
"886 :\n" \
120-
".option push\n" \
121-
".option norvc\n" \
122-
".option norelax\n" \
123-
old_c "\n" \
124-
".option pop\n" \
125-
"887 :\n" \
126-
ALT_NEW_CONTENT(vendor_id_1, errata_id_1, enable_1, new_c_1) \
112+
__ALTERNATIVE_CFG(old_c, new_c_1, vendor_id_1, errata_id_1, enable_1) \
127113
ALT_NEW_CONTENT(vendor_id_2, errata_id_2, enable_2, new_c_2)
128114

129115
#define _ALTERNATIVE_CFG_2(old_c, new_c_1, vendor_id_1, errata_id_1, \

0 commit comments

Comments
 (0)