public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH BlueZ v1] client: Fix gatt.{read, write} not working on scripts
@ 2025-07-21 20:09 Luiz Augusto von Dentz
  2025-07-21 21:32 ` [BlueZ,v1] " bluez.test.bot
  2025-07-23 14:40 ` [PATCH BlueZ v1] " patchwork-bot+bluetooth
  0 siblings, 2 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2025-07-21 20:09 UTC (permalink / raw)
  To: linux-bluetooth

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

If gatt.{read, write} are called within a script they don't indicate
completion so they block commands to be run after them.
---
 client/gatt.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/client/gatt.c b/client/gatt.c
index b18186518953..6dc80e2a31cd 100644
--- a/client/gatt.c
+++ b/client/gatt.c
@@ -696,6 +696,8 @@ void gatt_read_local_attribute(char *data, int argc, char *argv[])
 		bt_shell_hexdump(&d->value[offset], d->value_len - offset);
 		return bt_shell_noninteractive_quit(EXIT_SUCCESS);
 	}
+
+	return bt_shell_noninteractive_quit(EXIT_FAILURE);
 }
 
 static uint8_t *str2bytearray(char *arg, size_t *val_len)
@@ -809,6 +811,8 @@ void gatt_write_local_attribute(char *data, int argc, char *argv[])
 	}
 
 	free(value);
+
+	return bt_shell_noninteractive_quit(EXIT_SUCCESS);
 }
 
 static char *attribute_generator(const char *text, int state, GList *source)
-- 
2.50.1


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

* RE: [BlueZ,v1] client: Fix gatt.{read, write} not working on scripts
  2025-07-21 20:09 [PATCH BlueZ v1] client: Fix gatt.{read, write} not working on scripts Luiz Augusto von Dentz
@ 2025-07-21 21:32 ` bluez.test.bot
  2025-07-23 14:40 ` [PATCH BlueZ v1] " patchwork-bot+bluetooth
  1 sibling, 0 replies; 3+ messages in thread
From: bluez.test.bot @ 2025-07-21 21:32 UTC (permalink / raw)
  To: linux-bluetooth, luiz.dentz

[-- Attachment #1: Type: text/plain, Size: 1261 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=984445

---Test result---

Test Summary:
CheckPatch                    PENDING   0.29 seconds
GitLint                       PENDING   0.27 seconds
BuildEll                      PASS      20.49 seconds
BluezMake                     PASS      2636.17 seconds
MakeCheck                     PASS      20.59 seconds
MakeDistcheck                 PASS      189.93 seconds
CheckValgrind                 PASS      242.87 seconds
CheckSmatch                   PASS      315.63 seconds
bluezmakeextell               PASS      131.75 seconds
IncrementalBuild              PENDING   0.27 seconds
ScanBuild                     PASS      926.47 seconds

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

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

##############################
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 v1] client: Fix gatt.{read, write} not working on scripts
  2025-07-21 20:09 [PATCH BlueZ v1] client: Fix gatt.{read, write} not working on scripts Luiz Augusto von Dentz
  2025-07-21 21:32 ` [BlueZ,v1] " bluez.test.bot
@ 2025-07-23 14:40 ` patchwork-bot+bluetooth
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+bluetooth @ 2025-07-23 14:40 UTC (permalink / raw)
  To: Luiz Augusto von Dentz; +Cc: linux-bluetooth

Hello:

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

On Mon, 21 Jul 2025 16:09:33 -0400 you wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> 
> If gatt.{read, write} are called within a script they don't indicate
> completion so they block commands to be run after them.
> ---
>  client/gatt.c | 4 ++++
>  1 file changed, 4 insertions(+)

Here is the summary with links:
  - [BlueZ,v1] client: Fix gatt.{read, write} not working on scripts
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=06d73a43a854

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:[~2025-07-23 14:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-21 20:09 [PATCH BlueZ v1] client: Fix gatt.{read, write} not working on scripts Luiz Augusto von Dentz
2025-07-21 21:32 ` [BlueZ,v1] " bluez.test.bot
2025-07-23 14:40 ` [PATCH BlueZ v1] " 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