Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
From:   Vlad Pruteanu <vlad.pruteanu@nxp.com>
To:     linux-bluetooth@vger.kernel.org
Cc:     claudia.rosu@nxp.com, mihai-octavian.urzica@nxp.com,
        silviu.barbulescu@nxp.com, iulia.tanasescu@nxp.com,
        andrei.istodorescu@nxp.com, Vlad Pruteanu <vlad.pruteanu@nxp.com>
Subject: [PATCH v4 1/1] Bluetooth: ISO: Set CIS bit only for devices with CIS support
Date:   Tue, 12 Sep 2023 09:33:29 +0300
Message-Id: <20230912063329.1196184-2-vlad.pruteanu@nxp.com>
In-Reply-To: <20230912063329.1196184-1-vlad.pruteanu@nxp.com>
References: <20230912063329.1196184-1-vlad.pruteanu@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?jCPaUskuUrYSJ6D8EAsr158l9DfuS/W312PEV7zgIbVqJ2fatr5aDC/9zUbl?=
 =?us-ascii?Q?5COXVYXhgxRctr5mAFO7Yi7QhZOQTnV9M0zt44Th65F7gaTNBwj50M5TE0kq?=
 =?us-ascii?Q?BCflffWu9tpTfEsCZvOrRYXedR4Xwc8UXofXbuW8rU5Qz3CvlrugBhOh32rz?=
 =?us-ascii?Q?XNcABMJ2Qu18sKqF8wk9JKNNcGxoYCSZerTL4lnvThjBxXcCwZG7dETQ2epW?=
 =?us-ascii?Q?orTqgqADq1rNvu8XPxYJNBFtaCtp41eASGl/UaZ4QdIY+U46NAxUaabsXgUK?=
 =?us-ascii?Q?BmrnUaRzj4EQoiwzx6xTSWg6hfr3lU72DwhfwFbeCimFYrbfZ+8aYb3R1S25?=
 =?us-ascii?Q?FOL81YVYcPVv1pshdUc1jSjjdLOR+pPznpCpUCbPf5NL5GOOgJBvxCL8X6Lh?=
 =?us-ascii?Q?90MKLFzSXXJ5aGGbsnmVpxH23sTmPCE3qZV10WkOVSp1vLcMwjaDUEjdu7B7?=
 =?us-ascii?Q?h5bdPWgO0CXp5zuOATPUv4poGQpyVhB8CO2sr2dwdeu2jd4lErG03Ffy+m8Q?=
 =?us-ascii?Q?3ISX/2PdjHtiS01GSTVfsBbZUCXtF3yjspwGUdkjkS36sgjQIq2jf09P1BNI?=
 =?us-ascii?Q?KdDaxZnygP3N0hdc8BIQj7BXci1nVJL1rT0oSBpx0fnfEPDRsWhlxxew74Py?=
 =?us-ascii?Q?9Scl4dTMTJq0TK/QdnOMhizK/lJ3afUl1r0UlIWVy9NCCjr331Szjbonf5AG?=
 =?us-ascii?Q?RP91nwtUoVwzcnQcGbgyxg8AIYgOBek5T33SL4Xe5h53nu6UsTqPYwvD73Zk?=
 =?us-ascii?Q?KC6WIRo+5Qz9N2rhi19U0zJlHbAM2JMbAdxU6lNcQZ4fmvKgL5SxCTAPP5n8?=
 =?us-ascii?Q?2pnEqclNw0ArJ4tfxLMwd2MMBjhcmOJvrB/4HAdYroGwJACRTmHJNfuUmSm7?=
 =?us-ascii?Q?W5GBKFy18Frea1EgDwgHflOSG/RB0YGd5BFbsvsKjjTMY4aoRwVTFV6b47Ga?=
 =?us-ascii?Q?DfHSh05rWfzyqiotJobmxa+4Sc4ICgt2N5C7a9GzBOHwLjHdGtQVEqkIh5OE?=
 =?us-ascii?Q?0rG/tVZ7H9l29CPYM8JKfS/TU/Y5IrJhCRpbD+2sQoS6dgalPnD2OYx7I7IH?=
 =?us-ascii?Q?47icuyUe0Ot9f5khgnoxTZfmz6qjv5INiWmqqKis0U5QKVMjcn87YIi31eKN?=
 =?us-ascii?Q?oxBesydnEszH+VHKWFqxomP+v2pHVez6JOKy7b6XRLqEcn5J5r/FLrlpd2oi?=
 =?us-ascii?Q?rhVcwNTHpUiq4hVaXftfSyDkCvWaz/Sj7xM3mqZiHfItJoXN7qmjFQv4rmHY?=
 =?us-ascii?Q?vuaKc+gQJG27uFZPOt18VptrVcT2e7RV4QCiEoJL1/qUJ9KBCgBLBjkYEFxg?=
 =?us-ascii?Q?214wfQcA+QmHaCP/nJPY6KC3lSJXzyC9fnY/gLVxMQ+MFH+xvP/QN8+mxlcQ?=
 =?us-ascii?Q?DE4Md4jTFAJSgbWi7gTnVYtm43/yRMIcwNjXVDfQbt96SEsVl5y+POuXPcyo?=
 =?us-ascii?Q?m6hry2nRLW+K3S0ZFVdG6IhDlH601AR89lj4yl9KDuZBO7A9koeXyyoJ1fxu?=
 =?us-ascii?Q?tg5TKvPiL1AOLmebJwK70BjpEPMLqoXF52mwGPaGqgs1wNZL0egyLqP3R3QE?=
 =?us-ascii?Q?tnYRL224+fDWcEZwb4mmicJ3uEaFwIySzH+WhzmeOJkzVjlBmPeP7Rw+3iD4?=
 =?us-ascii?Q?BA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 03ef54ac-c447-43f0-d1b2-08dbb35a4202
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9398.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Sep 2023 06:34:04.0909
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ZkHhHL2L8c9H9do+tnvFmDPIDBOOca9wSyhawUeU3rZFgLCmgYH9NHcaC6iEr365OXL4nQ7sZStt4DneJo5sMQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8359
Precedence: bulk
List-ID: <linux-bluetooth.vger.kernel.org>
X-Mailing-List: linux-bluetooth@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Mon, 11 Sep 2023 23:34:09 -0700 (PDT)
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email

Currently the CIS bit that can be set by the host is set for any device
that has CIS or BIS support. In reality, devices that support BIS may not
allow that bit to be set and so, the HCI bring up fails for them.

This commit fixes this by only setting the bit for CIS capable devices.

Signed-off-by: Vlad Pruteanu <vlad.pruteanu@nxp.com>
---
 net/bluetooth/hci_sync.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c
index ec4dfc4c5749..1be5763c261c 100644
--- a/net/bluetooth/hci_sync.c
+++ b/net/bluetooth/hci_sync.c
@@ -4264,12 +4264,12 @@ static int hci_le_set_host_feature_sync(struct hci_dev *hdev)
 {
 	struct hci_cp_le_set_host_feature cp;
 
-	if (!iso_capable(hdev))
+	if (!cis_capable(hdev))
 		return 0;
 
 	memset(&cp, 0, sizeof(cp));
 
-	/* Isochronous Channels (Host Support) */
+	/* Connected Isochronous Channels (Host Support) */
 	cp.bit_number = 32;
 	cp.bit_value = 1;
 
-- 
2.34.1