Message ID | 1453934861-26364-3-git-send-email-thomas.petazzoni@free-electrons.com |
---|---|
State | Superseded |
Headers | show |
Thomas, All, On 2016-01-27 23:47 +0100, Thomas Petazzoni spake thusly: > This commit updates the documentation to detail when and how to use > the BR2_TOOLCHAIN_HAS_SYNC_x dependency. Note that we chose to not add > a comment about this dependency, because it is mainly tied to > architecture capabilities (except in very specific cases, which would > be way too complicated to explain in a Config.in comment). > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > --- > docs/manual/adding-packages-directory.txt | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/docs/manual/adding-packages-directory.txt b/docs/manual/adding-packages-directory.txt > index b9596a2..5329c10 100644 > --- a/docs/manual/adding-packages-directory.txt > +++ b/docs/manual/adding-packages-directory.txt > @@ -279,6 +279,16 @@ use in the comment. > ** Dependency symbol: +BR2_ARCH_HAS_ATOMICS+ > ** Comment string: no comment to be added > > +* Gcc +__sync_*+ built-ins used for atomic operations. They are > + available in variants operating on 1 byte, 2 bytes, 4 bytes and 8 > + bytes. Since different architectures support atomic operations on > + different sizes, one dependency symbol is available for each size: > +** Dependency symbol: +BR2_TOOLCHAIN_HAS_SYNC_1+ for 1 byte, > + +BR2_TOOLCHAIN_HAS_SYNC_2+ for two bytes, Why 'two' in letters here, when you otherwise have 1, 4 and 8 in numerals? Otherwise: Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Regards, Yann E. MORIN. > + +BR2_TOOLCHAIN_HAS_SYNC_4+ for 4 bytes, +BR2_TOOLCHAIN_HAS_SYNC_8+ > + for 8 bytes. > +** Comment string: no comment to be added > + > * Kernel headers > ** Dependency symbol: +BR2_TOOLCHAIN_HEADERS_AT_LEAST_X_Y+, (replace > +X_Y+ with the proper version, see +toolchain/toolchain-common.in+) > -- > 2.6.4 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
diff --git a/docs/manual/adding-packages-directory.txt b/docs/manual/adding-packages-directory.txt index b9596a2..5329c10 100644 --- a/docs/manual/adding-packages-directory.txt +++ b/docs/manual/adding-packages-directory.txt @@ -279,6 +279,16 @@ use in the comment. ** Dependency symbol: +BR2_ARCH_HAS_ATOMICS+ ** Comment string: no comment to be added +* Gcc +__sync_*+ built-ins used for atomic operations. They are + available in variants operating on 1 byte, 2 bytes, 4 bytes and 8 + bytes. Since different architectures support atomic operations on + different sizes, one dependency symbol is available for each size: +** Dependency symbol: +BR2_TOOLCHAIN_HAS_SYNC_1+ for 1 byte, + +BR2_TOOLCHAIN_HAS_SYNC_2+ for two bytes, + +BR2_TOOLCHAIN_HAS_SYNC_4+ for 4 bytes, +BR2_TOOLCHAIN_HAS_SYNC_8+ + for 8 bytes. +** Comment string: no comment to be added + * Kernel headers ** Dependency symbol: +BR2_TOOLCHAIN_HEADERS_AT_LEAST_X_Y+, (replace +X_Y+ with the proper version, see +toolchain/toolchain-common.in+)
This commit updates the documentation to detail when and how to use the BR2_TOOLCHAIN_HAS_SYNC_x dependency. Note that we chose to not add a comment about this dependency, because it is mainly tied to architecture capabilities (except in very specific cases, which would be way too complicated to explain in a Config.in comment). Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> --- docs/manual/adding-packages-directory.txt | 10 ++++++++++ 1 file changed, 10 insertions(+)