linux-kernel - Re: [PATCH v1] Bluetooth: btmtksdio: enable msft opcode

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 30 Nov 2021 13:01:32 +0000
From:   Łukasz Bartosik <lb@...ihalf.com>
To:     Sean Wang <sean.wang@...iatek.com>
Cc:     Marcel Holtmann <marcel@...tmann.org>,
        Johan Hedberg <johan.hedberg@...il.com>,
        Luiz Augusto von Dentz <luiz.dentz@...il.com>,
        upstream@...ihalf.com, linux-bluetooth@...r.kernel.org,
        linux-mediatek@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] Bluetooth: btmtksdio: enable msft opcode

pon., 29 lis 2021 o 22:01 <sean.wang@...iatek.com> napisał(a):
>
> From: Sean Wang <sean.wang@...iatek.com>
>
> >Enable msft opcode for btmtksdio driver.
> >
> >Signed-off-by: Łukasz Bartosik <lb@...ihalf.com>
> >---
> > drivers/bluetooth/btmtksdio.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> >diff --git a/drivers/bluetooth/btmtksdio.c b/drivers/bluetooth/btmtksdio.c index d9cf0c492e29..2a7a615663b9 100644
> >--- a/drivers/bluetooth/btmtksdio.c
> >+++ b/drivers/bluetooth/btmtksdio.c
> >@@ -887,6 +887,7 @@ static int btmtksdio_setup(struct hci_dev *hdev)
> >       if (enable_autosuspend)
> >               pm_runtime_allow(bdev->dev);
> >
> >+      hci_set_msft_opcode(hdev, 0xFD30);
>
> Hi Łukasz,
>
> msft feature is supposed only supported on mt7921. Could you help rework the patch to enalbe msft opocde only for mt7921?
>
>         Sean
>

Hi Sean,

Thanks for the input. I will update the patch so that msft opcode is
enabled only for mt7921.

Regards,
Lukasz

> >       bt_dev_info(hdev, "Device setup in %llu usecs", duration);
> >
> >       return 0;
> >