* [PATCH] bluetooth: Fix op_code size entry from the previous patch
@ 2020-06-12 19:33 Alain Michaud
2020-06-12 19:42 ` Marcel Holtmann
0 siblings, 1 reply; 3+ messages in thread
From: Alain Michaud @ 2020-06-12 19:33 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Alain Michaud
The previous applied patch introduced an error that was not present in
the submitted patch here: https://patchwork.kernel.org/patch/11599123/
In particular MGMT_SET_DEF_SYSTEM_CONFIG_SIZE was introduced but
MGMT_SET_DEF_SYSTEM_CONFIG was used.
Signed-off-by: Alain Michaud <alainm@chromium.org>
---
net/bluetooth/mgmt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
index 2e0f976e7e04..99fbfd467d04 100644
--- a/net/bluetooth/mgmt.c
+++ b/net/bluetooth/mgmt.c
@@ -7303,7 +7303,7 @@ static const struct hci_mgmt_handler mgmt_handlers[] = {
HCI_MGMT_HDEV_OPTIONAL },
{ read_def_system_config, MGMT_READ_DEF_SYSTEM_CONFIG_SIZE,
HCI_MGMT_UNTRUSTED },
- { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG,
+ { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG_SIZE,
HCI_MGMT_VAR_LEN },
};
--
2.27.0.290.gba653c62da-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] bluetooth: Fix op_code size entry from the previous patch
2020-06-12 19:33 [PATCH] bluetooth: Fix op_code size entry from the previous patch Alain Michaud
@ 2020-06-12 19:42 ` Marcel Holtmann
2020-06-12 19:51 ` Alain Michaud
0 siblings, 1 reply; 3+ messages in thread
From: Marcel Holtmann @ 2020-06-12 19:42 UTC (permalink / raw)
To: Alain Michaud; +Cc: linux-bluetooth
Hi Alain,
> The previous applied patch introduced an error that was not present in
> the submitted patch here: https://patchwork.kernel.org/patch/11599123/
>
> In particular MGMT_SET_DEF_SYSTEM_CONFIG_SIZE was introduced but
> MGMT_SET_DEF_SYSTEM_CONFIG was used.
>
> Signed-off-by: Alain Michaud <alainm@chromium.org>
> ---
>
> net/bluetooth/mgmt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> index 2e0f976e7e04..99fbfd467d04 100644
> --- a/net/bluetooth/mgmt.c
> +++ b/net/bluetooth/mgmt.c
> @@ -7303,7 +7303,7 @@ static const struct hci_mgmt_handler mgmt_handlers[] = {
> HCI_MGMT_HDEV_OPTIONAL },
> { read_def_system_config, MGMT_READ_DEF_SYSTEM_CONFIG_SIZE,
> HCI_MGMT_UNTRUSTED },
> - { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG,
> + { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG_SIZE,
> HCI_MGMT_VAR_LEN },
This was me being stupid. I had this fixed up in my tree, but forgot to amend the commit. I force pushed a correct commit to bluetooth-next now.
Regards
Marcel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] bluetooth: Fix op_code size entry from the previous patch
2020-06-12 19:42 ` Marcel Holtmann
@ 2020-06-12 19:51 ` Alain Michaud
0 siblings, 0 replies; 3+ messages in thread
From: Alain Michaud @ 2020-06-12 19:51 UTC (permalink / raw)
To: Marcel Holtmann; +Cc: Alain Michaud, BlueZ
OK, thanks!
On Fri, Jun 12, 2020 at 3:42 PM Marcel Holtmann <marcel@holtmann.org> wrote:
>
> Hi Alain,
>
> > The previous applied patch introduced an error that was not present in
> > the submitted patch here: https://patchwork.kernel.org/patch/11599123/
> >
> > In particular MGMT_SET_DEF_SYSTEM_CONFIG_SIZE was introduced but
> > MGMT_SET_DEF_SYSTEM_CONFIG was used.
> >
> > Signed-off-by: Alain Michaud <alainm@chromium.org>
> > ---
> >
> > net/bluetooth/mgmt.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> > index 2e0f976e7e04..99fbfd467d04 100644
> > --- a/net/bluetooth/mgmt.c
> > +++ b/net/bluetooth/mgmt.c
> > @@ -7303,7 +7303,7 @@ static const struct hci_mgmt_handler mgmt_handlers[] = {
> > HCI_MGMT_HDEV_OPTIONAL },
> > { read_def_system_config, MGMT_READ_DEF_SYSTEM_CONFIG_SIZE,
> > HCI_MGMT_UNTRUSTED },
> > - { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG,
> > + { set_def_system_config, MGMT_SET_DEF_SYSTEM_CONFIG_SIZE,
> > HCI_MGMT_VAR_LEN },
>
> This was me being stupid. I had this fixed up in my tree, but forgot to amend the commit. I force pushed a correct commit to bluetooth-next now.
>
> Regards
>
> Marcel
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-12 19:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-12 19:33 [PATCH] bluetooth: Fix op_code size entry from the previous patch Alain Michaud
2020-06-12 19:42 ` Marcel Holtmann
2020-06-12 19:51 ` Alain Michaud
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.