<feed xmlns='http://www.w3.org/2005/Atom'>
<title>opensbi.git/Makefile, branch release-1.5.x</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/opensbi.git/atom/Makefile?h=release-1.5.x</id>
<link rel='self' href='http://cgit.235523.xyz/opensbi.git/atom/Makefile?h=release-1.5.x'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/'/>
<updated>2024-04-08T04:38:20Z</updated>
<entry>
<title>Makefile: Remove unnecessary dependencies</title>
<updated>2024-04-08T04:38:20Z</updated>
<author>
<name>Samuel Holland</name>
<email>samuel.holland@sifive.com</email>
</author>
<published>2024-03-18T20:49:41Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=a7c5c2cbd27af430e80d894944c479e5e4df71b3'/>
<id>urn:sha1:a7c5c2cbd27af430e80d894944c479e5e4df71b3</id>
<content type='text'>
The rule included from auto.conf.cmd adds a dependency on every Kconfig
file, so these two Kconfig files do not need to be specified again here.

Signed-off-by: Samuel Holland &lt;samuel.holland@sifive.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
Tested-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: Respect manual changes to .config</title>
<updated>2024-04-07T06:35:37Z</updated>
<author>
<name>Samuel Holland</name>
<email>samuel.holland@sifive.com</email>
</author>
<published>2024-03-18T20:49:40Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=268feab2949a08743ffbb4afd9d40cdcafd78376'/>
<id>urn:sha1:268feab2949a08743ffbb4afd9d40cdcafd78376</id>
<content type='text'>
The .config file may be manually edited or copied from another location.
Since genconfig.py is responsible for generating auto.conf (the Makefile
fragment) and autoconf.h (the C header) from .config, it must be run any
time .config changes, not just when running menuconfig.

Fixes: 662e631ccef2 ("Makefile: Add initial kconfig support for each platform")
Signed-off-by: Samuel Holland &lt;samuel.holland@sifive.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
Tested-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>firmware: remove copy-base relocation</title>
<updated>2024-04-05T09:35:25Z</updated>
<author>
<name>Xiang W</name>
<email>wxjstz@126.com</email>
</author>
<published>2024-03-12T08:24:59Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=76d7e9b8ee72c226cd8424d5483827bcf4c792fa'/>
<id>urn:sha1:76d7e9b8ee72c226cd8424d5483827bcf4c792fa</id>
<content type='text'>
Remove copy-base relocations that are no longer needed.

Signed-off-by: Xiang W &lt;wxjstz@126.com&gt;
Reviewed-by: Samuel Holland &lt;samuel.holland@sifive.com&gt;
Tested-by: Samuel Holland &lt;samuel.holland@sifive.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: check for --exclude-libs</title>
<updated>2024-02-05T11:04:44Z</updated>
<author>
<name>Leon M. Busch-George</name>
<email>leon@georgemail.eu</email>
</author>
<published>2024-01-26T15:21:24Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=d1dad07cb88fbf601556b4df8ef2bac70fed7143'/>
<id>urn:sha1:d1dad07cb88fbf601556b4df8ef2bac70fed7143</id>
<content type='text'>
While writing to the dynsym is futile, the --exclude-libs options is not
recognized by all linkers (e.g. riscv64-elf-ld.bfd).

Signed-off-by: Leon M. Busch-George &lt;leon@georgemail.eu&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: don't pass -mstrict-align if not supported</title>
<updated>2024-02-05T10:30:54Z</updated>
<author>
<name>Kalle Wachsmuth</name>
<email>kalle.wachsmuth@gmail.com</email>
</author>
<published>2024-01-18T18:02:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=4a76f79ff5e01bb72eedcf636a11f4c701d1568d'/>
<id>urn:sha1:4a76f79ff5e01bb72eedcf636a11f4c701d1568d</id>
<content type='text'>
Support for that option will be added in LLVM 18:
https://github.com/llvm/llvm-project/commit/23ce5368409c760f3dd49d0f17f34772b0b869d8

Clang 17.0.6, however, will error when passed the
`-mstrict-align` flag.
We should only use the flag if it is supported.

Signed-off-by: Kalle Wachsmuth &lt;kalle.wachsmuth@gmail.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
Reviewed-by: Xiang W &lt;wxjstz@126.com&gt;
</content>
</entry>
<entry>
<title>Makefile: Add --exclude-libs ALL to avoid .dynsym</title>
<updated>2023-10-06T03:51:24Z</updated>
<author>
<name>Vivian Wang</name>
<email>dramforever@live.com</email>
</author>
<published>2023-09-17T17:52:51Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=6ed125a6027eea0f22b92d96d6fb538446fa94c0'/>
<id>urn:sha1:6ed125a6027eea0f22b92d96d6fb538446fa94c0</id>
<content type='text'>
Since everything is statically linked, we don't need to expose symbols
for dynamic linking.

For a default build this saves about 2 KiB of useless read only data in
.dynsym, .dynstr, .hash, .gnu.hash sections.

Signed-off-by: Vivian Wang &lt;dramforever@live.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: Fix grep warning</title>
<updated>2023-08-22T07:56:03Z</updated>
<author>
<name>Andrew Jones</name>
<email>ajones@ventanamicro.com</email>
</author>
<published>2023-08-10T12:15:28Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=7aabeee93edb2e7ab03954e235683bfd9d8f29eb'/>
<id>urn:sha1:7aabeee93edb2e7ab03954e235683bfd9d8f29eb</id>
<content type='text'>
grep (at least my version, grep-3.8-3.fc38.x86_64) warns with
"grep: warning: stray \ before -". Fix the warning by making
the command line input to grep less ambiguous.

Signed-off-by: Andrew Jones &lt;ajones@ventanamicro.com&gt;
Reviewed-by: Anup Patel  &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: Dereference symlinks on install</title>
<updated>2023-05-22T03:22:57Z</updated>
<author>
<name>Filip Filmar</name>
<email>fmil@google.com</email>
</author>
<published>2023-05-16T02:38:45Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=d4c46e0ff1b0ead9d5a586e1a19a00a92160206d'/>
<id>urn:sha1:d4c46e0ff1b0ead9d5a586e1a19a00a92160206d</id>
<content type='text'>
Adds the `-L` flag (follow symlinks) to the `cp` commands used to
install `libsbi.a` and `include/sbi/*`.

This should make no difference in regular compilation. However,
it does make a difference when compiling with bazel.  Namely,
bazel's sandboxing will turn all the source files into symlinks.
After installation with `cp` the destination files will be
symlinks pointing to the sandbox symlinks. As the sandbox files
are removed when compilation ends, the just-copied symlinks
become dangling symlinks.

The resulting include files will be
unusable due to the dangling symlink issues. Adding `-L` when
copying ensures that the files obtained by executing the `install`
targets are always dereferenced to files, rather than symlinks,
eliminating this issue.

Signed-off-by: Filip Filmar &lt;fmil@google.com&gt;
Reviewed-by: Xiang W &lt;wxjstz@126.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>make: Add a command line option for debugging OpenSBI</title>
<updated>2023-03-01T03:53:17Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng@tinylab.org</email>
</author>
<published>2023-02-27T02:35:06Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=4b28afc98bbe406e3ad6f4a97d0fe96a882e83a1'/>
<id>urn:sha1:4b28afc98bbe406e3ad6f4a97d0fe96a882e83a1</id>
<content type='text'>
Add a new make command line option "make DEBUG=1" to prevent compiler
optimizations using -O2.

Signed-off-by: Bin Meng &lt;bmeng@tinylab.org&gt;
Reviewed-by: Xiang W &lt;wxjstz@126.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
<entry>
<title>Makefile: Add missing .dep files for fw_*.elf.ld</title>
<updated>2023-02-08T11:06:55Z</updated>
<author>
<name>Jessica Clarke</name>
<email>jrtc27@jrtc27.com</email>
</author>
<published>2023-02-01T16:49:16Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/opensbi.git/commit/?id=642f3de9b947b747a4fe0eaae348f1c55f1ac409'/>
<id>urn:sha1:642f3de9b947b747a4fe0eaae348f1c55f1ac409</id>
<content type='text'>
Since we don't currently create these, changes to fw_base.ldS do not
cause the preprocessed fw_*.elf.ld files to be rebuilt, and thus
incremental builds can end up failing with missing symbols if crossing
the recent commits that introduced _fw_rw_offset and then replaced it
with _fw_rw_start.

Reported-by: Ben Dooks &lt;ben.dooks@sifive.com&gt;
Signed-off-by: Jessica Clarke &lt;jrtc27@jrtc27.com&gt;
Reviewed-by: Anup Patel &lt;anup@brainfault.org&gt;
</content>
</entry>
</feed>
