public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH BlueZ v2] emulator: Fix compilation on big endian systems
@ 2026-02-10  9:32 Bastien Nocera
  2026-02-10 11:18 ` [BlueZ,v2] " bluez.test.bot
  2026-02-10 18:00 ` [PATCH BlueZ v2] " patchwork-bot+bluetooth
  0 siblings, 2 replies; 3+ messages in thread
From: Bastien Nocera @ 2026-02-10  9:32 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Pauli Virtanen

This fixes the build on big endian systems like s390x:

Remove the "static" so the array doesn't need to be initialised
with constants.

emulator/bthost.c: In function ‘bthost_setup_sco’:
./src/shared/util.h:43:26: error: initializer element is not constant
   43 | #define cpu_to_le32(val) bswap_32(val)
      |                          ^~~~~~~~
etc.

Fixes: 85888a8357ea ("bthost: add bthost_setup_sco() and accept
incoming eSCO")
---
 emulator/bthost.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/emulator/bthost.c b/emulator/bthost.c
index d09ad1e76c50..53b12f828675 100644
--- a/emulator/bthost.c
+++ b/emulator/bthost.c
@@ -3560,7 +3560,7 @@ void bthost_hci_disconnect(struct bthost *bthost, uint16_t handle,
 int bthost_setup_sco(struct bthost *bthost, uint16_t acl_handle,
 							uint16_t setting)
 {
-	static const struct bt_hci_cmd_setup_sync_conn settings[] = {
+	const struct bt_hci_cmd_setup_sync_conn settings[] = {
 		{
 			.tx_bandwidth = cpu_to_le32(0x00001f40),
 			.rx_bandwidth = cpu_to_le32(0x00001f40),
-- 
2.52.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* RE: [BlueZ,v2] emulator: Fix compilation on big endian systems
  2026-02-10  9:32 [PATCH BlueZ v2] emulator: Fix compilation on big endian systems Bastien Nocera
@ 2026-02-10 11:18 ` bluez.test.bot
  2026-02-10 18:00 ` [PATCH BlueZ v2] " patchwork-bot+bluetooth
  1 sibling, 0 replies; 3+ messages in thread
From: bluez.test.bot @ 2026-02-10 11:18 UTC (permalink / raw)
  To: linux-bluetooth, hadess

[-- Attachment #1: Type: text/plain, Size: 1689 bytes --]

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=1052631

---Test result---

Test Summary:
CheckPatch                    PENDING   0.51 seconds
GitLint                       PENDING   0.54 seconds
BuildEll                      PASS      20.30 seconds
BluezMake                     PASS      658.81 seconds
MakeCheck                     PASS      18.54 seconds
MakeDistcheck                 PASS      251.17 seconds
CheckValgrind                 PASS      300.09 seconds
CheckSmatch                   WARNING   359.76 seconds
bluezmakeextell               PASS      187.40 seconds
IncrementalBuild              PENDING   0.45 seconds
ScanBuild                     PASS      1061.52 seconds

Details
##############################
Test: CheckPatch - PENDING
Desc: Run checkpatch.pl script
Output:

##############################
Test: GitLint - PENDING
Desc: Run gitlint
Output:

##############################
Test: CheckSmatch - WARNING
Desc: Run smatch tool with source
Output:
emulator/bthost.c:700:28: warning: Variable length array is used.emulator/bthost.c:701:32: warning: Variable length array is used.emulator/bthost.c:918:28: warning: Variable length array is used.emulator/bthost.c:952:28: warning: Variable length array is used.emulator/bthost.c:953:32: warning: Variable length array is used.
##############################
Test: IncrementalBuild - PENDING
Desc: Incremental build with the patches in the series
Output:



---
Regards,
Linux Bluetooth


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH BlueZ v2] emulator: Fix compilation on big endian systems
  2026-02-10  9:32 [PATCH BlueZ v2] emulator: Fix compilation on big endian systems Bastien Nocera
  2026-02-10 11:18 ` [BlueZ,v2] " bluez.test.bot
@ 2026-02-10 18:00 ` patchwork-bot+bluetooth
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+bluetooth @ 2026-02-10 18:00 UTC (permalink / raw)
  To: Bastien Nocera; +Cc: linux-bluetooth, pav

Hello:

This patch was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Tue, 10 Feb 2026 10:32:47 +0100 you wrote:
> This fixes the build on big endian systems like s390x:
> 
> Remove the "static" so the array doesn't need to be initialised
> with constants.
> 
> emulator/bthost.c: In function ‘bthost_setup_sco’:
> ./src/shared/util.h:43:26: error: initializer element is not constant
>    43 | #define cpu_to_le32(val) bswap_32(val)
>       |                          ^~~~~~~~
> etc.
> 
> [...]

Here is the summary with links:
  - [BlueZ,v2] emulator: Fix compilation on big endian systems
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=f1eedda9cefc

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-02-10 18:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-10  9:32 [PATCH BlueZ v2] emulator: Fix compilation on big endian systems Bastien Nocera
2026-02-10 11:18 ` [BlueZ,v2] " bluez.test.bot
2026-02-10 18:00 ` [PATCH BlueZ v2] " patchwork-bot+bluetooth

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox