2021-03-16 15:37:03

by Ayush Garg

Subject: [PATCH BlueZ] btmgmt: Fix typo in printing Set PHY Configuration command error

Signed-off-by: Ayush Garg <[email protected]>
---
tools/btmgmt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/btmgmt.c b/tools/btmgmt.c
index f4eb541fa..647c696f8 100644
--- a/tools/btmgmt.c
+++ b/tools/btmgmt.c
@@ -4725,7 +4725,7 @@ static void cmd_phy(int argc, char **argv)
if (mgmt_send(mgmt, MGMT_OP_SET_PHY_CONFIGURATION, index, sizeof(cp),
&cp, set_phy_rsp, NULL, NULL) == 0) {
error("Unable to send %s cmd",
- mgmt_opstr(MGMT_OP_GET_PHY_CONFIGURATION));
+ mgmt_opstr(MGMT_OP_SET_PHY_CONFIGURATION));
return bt_shell_noninteractive_quit(EXIT_FAILURE);
}
}
--
2.17.1

2021-03-16 18:50:30

by bluez.test.bot

Subject: RE: [BlueZ] btmgmt: Fix typo in printing Set PHY Configuration command error

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=448991

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuild - PASS

##############################
Test: MakeCheck - PASS



---
Regards,
Linux Bluetooth

2021-03-16 21:20:25

by Luiz Augusto von Dentz

Subject: Re: [PATCH BlueZ] btmgmt: Fix typo in printing Set PHY Configuration command error

Hi Ayush,

On Tue, Mar 16, 2021 at 8:37 AM Ayush Garg <[email protected]> wrote:
>
> Signed-off-by: Ayush Garg <[email protected]>
> ---
> tools/btmgmt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/btmgmt.c b/tools/btmgmt.c
> index f4eb541fa..647c696f8 100644
> --- a/tools/btmgmt.c
> +++ b/tools/btmgmt.c
> @@ -4725,7 +4725,7 @@ static void cmd_phy(int argc, char **argv)
> if (mgmt_send(mgmt, MGMT_OP_SET_PHY_CONFIGURATION, index, sizeof(cp),
> &cp, set_phy_rsp, NULL, NULL) == 0) {
> error("Unable to send %s cmd",
> - mgmt_opstr(MGMT_OP_GET_PHY_CONFIGURATION));
> + mgmt_opstr(MGMT_OP_SET_PHY_CONFIGURATION));
> return bt_shell_noninteractive_quit(EXIT_FAILURE);
> }
> }
> --
> 2.17.1

Applied, note that I changed the commit message a little bit and
removed the Signed-off-by since we don't use that in userspace.

--
Luiz Augusto von Dentz