Received: by 2002:a05:7412:2a91:b0:fc:a2b0:25d7 with SMTP id u17csp24474rdh; Tue, 13 Feb 2024 08:18:25 -0800 (PST) X-Google-Smtp-Source: AGHT+IGwJVp7C0IcHoM4ziZYK6JolbBGGdH7MPE2Cjy9Bav6oDViRNmVJIxv9qt7iCugoiJ+aVf7 X-Received: by 2002:a05:6402:68b:b0:55f:52c5:ab9d with SMTP id f11-20020a056402068b00b0055f52c5ab9dmr84916edy.28.1707841105715; Tue, 13 Feb 2024 08:18:25 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707841105; cv=pass; d=google.com; s=arc-20160816; b=NLUsaMVkRchthxOJPFMJlTnJNQDP2d5ZsWLfwxQikjb8/d0dBUkv65+L0cksRNwkft BtsR3rfnaqbCLL58AwCyQ8sv+pwlsk91tcl/MBTHa6DhwPb8vFOhcoEebfmnw1VL3W1y L0kShyOEBcDp4TVA0J1NPhFqaZYIvb0vZV8iypSTrm7pkktHQFsLqkb6yBINxAHL54zT CzHA+pCEM14ar96nEU4mqrHiZTXdbbZJvLQSvP4q9y78ZjqM3NTCzvfcCFJ3JKUjzBTF q5CiBxluVGmyMVomZzYcwIJrg8p2wKWP4qJXX+y/i2izaI2cA8dp0LjMDL/0Nv/laORT 69jQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=FlgJF35WGDHbKHgYXjs8y1mAvdUenBmv+lFnZes+sKc=; fh=aYNrjar/iPKCe64wDx/EolT6QBoGv9OmiHmgcTAdtcU=; b=Byz6aRQHmVCMGKoYqwrQM0MHQk+E7QUPd2vGKNzPpYC9aJ+gfc1smXfYZdjYytrC9u R0J7Iv038Cpd6N9IejVQc4di6S6aeksQZTp5vvpKxSDXCUsJoEVIpyhIm1lPbeLduoLp dc99ZbG+MVFDnlsaqQ438f4daL8EugrFFecbyzei1N4Wt5+X8u5oT0Vzi+VZPOfvYltC oosJTV3gKTRVgZoJiQnxYGDhPvUOGJgd3F4M/OVmPae9RfkE33jQvmBGuOL2MP3uC2nF gdj9ydY0p+UOieIeVnaF11FOo/ykl47QuZuMF48brn9mQLIP5olX83KnMu/j929zFwi3 CKEA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b="FpqH/Sjf"; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-bluetooth+bounces-1827-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-bluetooth+bounces-1827-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org X-Forwarded-Encrypted: i=2; AJvYcCVQ7G5Yed/+DGY4zStl8f8ZZpQboI3rxkgm9lpvseO+1m++c0VRJjqpN7MhBklIPdx2rwe3ZnlZpPRs7MlwM2K5vz/helIjc7QBq+Do8Q== Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id j5-20020aa7c405000000b00560c92f8706si3777266edq.576.2024.02.13.08.18.25 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Feb 2024 08:18:25 -0800 (PST) Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-1827-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b="FpqH/Sjf"; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-bluetooth+bounces-1827-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-bluetooth+bounces-1827-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 4E7231F252BF for ; Tue, 13 Feb 2024 16:18:25 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 806125F840; Tue, 13 Feb 2024 16:18:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="FpqH/Sjf" X-Original-To: linux-bluetooth@vger.kernel.org Received: from mail-io1-f42.google.com (mail-io1-f42.google.com [209.85.166.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 53C0A5F846 for ; Tue, 13 Feb 2024 16:18:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707841093; cv=none; b=c1pUCn/FELTWO5ItQrX0D99y7Mm8pXw41QYzvInx3LrudTkq5tP2eOtXHPpEil2o0Aqv5i8OM/YA1qtW/pZk7L/kf/iUDbzvjaknYIkiIWk854E1ij6CO6apZJo/CR51Dxd1t9f4ljBLH5lmq/2j2snjjWdNgPnwXRqS/TXxPEs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707841093; c=relaxed/simple; bh=2KcI4FrqWW+CE3taGQPv55SZsbQRxd4d/ksvlG124dw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=p8D8FBgnLpV1hnEBs+n38Z/fVPmx7rWolGMGi+QGbpFdjOK2WieDo9LZd+eWk+iazwnfHRC+z5/IgmhdvlptaF15se1EiPb/X1Gp7OTo0DfjmAepOKvewfPzcOG1aXhiR4XvUzIz4BBHBj6RifqZssMmkmYqZs6GMxrYEPbHfo4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=FpqH/Sjf; arc=none smtp.client-ip=209.85.166.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Received: by mail-io1-f42.google.com with SMTP id ca18e2360f4ac-7c47f06863bso6283539f.3 for ; Tue, 13 Feb 2024 08:18:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1707841091; x=1708445891; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=FlgJF35WGDHbKHgYXjs8y1mAvdUenBmv+lFnZes+sKc=; b=FpqH/SjfLTQOhNbppyXemHAfStWs67TFI3G/6FeshxbC/17jQ2/Ohiw5sxNfiRELFK 6C4LaFHNis+aU18MFY4BlMy2D7hSZruDT+w4RZC/QPRWD4idpk4Yj1Yh4rMw+gMRJGy/ gTWho6ehAyFnB9O1QAXGCEbaDV/ppAmuJNfRM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707841091; x=1708445891; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=FlgJF35WGDHbKHgYXjs8y1mAvdUenBmv+lFnZes+sKc=; b=urLpkmK5fbQ0gaqLdP7hEyEDbPBv4aEs19ef3Eakxx/GeuIT0aWrEE9xJcita5cbFg LXYVNwGh83WPwbxE2yUvghMlPnW6xStTpXcK8bohsfZqeFz+oZKkykpONmD4Ot/q3clt PO8TFaOwf4Pzr1+8djY89NAg0wHW2SBRFPb7Dh66dq5Kt3WFYtk51K558efeOeqldiFc oDP/v/UreeOIf3FzU+DJPbdaqZnfhC9y/BudStmeGyAnDnpT+qR/KZZLK3ejvvOZG4RY qbiiMyhUo4h9aMEVkD6Agzi+r2EsFxGL/wmcvePgPXlF3KIDSAkeCPKf3uea4vUawDwy V7rg== X-Gm-Message-State: AOJu0YyjsVdBD0zoJVvjdP8zwvKMbpHe+MW7okkuIcRP2bEjEqt3+cWS NRmlckH385Y2Olgln8dVHfYwtgaRX16TC/ec4Juapq+b6FSAoea93Si4nmM0JQ== X-Received: by 2002:a6b:7515:0:b0:7c4:5e08:f265 with SMTP id l21-20020a6b7515000000b007c45e08f265mr125289ioh.13.1707841091396; Tue, 13 Feb 2024 08:18:11 -0800 (PST) X-Forwarded-Encrypted: i=1; AJvYcCXiwgCA4mecHx4KE5dEq/TSP0Bxd3qlRWtDMUGg+EfGw7CyyT1YC8SSI7YvVxApPQemo6zI49wIsV+VeDX6naVJNvBnD5+enqNqtnwtVMrQf7mq8x+lsEA+yskH6ne9hi+zZiRKPT5Sycs3wHf+UPoLzwOTqqG0tSeFAsHJzmartE8XIs5l71yirgpSmyk0t8OBOGk33na345zbpCHvL8rt6KppuqiwrozHn31CDAs8DSwSIYi5VGZc9x/wYaMdVCVfbmxWFn3BNHnDVxZsi5ZG+ZGCPL5ouFmvS1eevS2WZQUF37OHlhVICnhNMv4hBbLflCxd8JWxhCbbyfHoJrdln8YyWEs34+GUlKWrMas8J0ssUcZ6i6bRzZx56ucCKTLhhDnogFnjpxbLQHdTDsoObiqXh+dYuCpuqqyXmxvVD76ZDDeF9b9va1Pksx1Ov+0EStnZiSsjnixI9wFG4g== Received: from localhost (147.220.222.35.bc.googleusercontent.com. [35.222.220.147]) by smtp.gmail.com with UTF8SMTPSA id f26-20020a02cada000000b00471346c4225sm1922415jap.7.2024.02.13.08.18.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 13 Feb 2024 08:18:10 -0800 (PST) Date: Tue, 13 Feb 2024 16:18:10 +0000 From: Matthias Kaehlcke To: Johan Hovold Cc: Luiz Augusto von Dentz , Marcel Holtmann , Johan Hedberg , Bjorn Andersson , Konrad Dybcio , linux-bluetooth@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Balakrishna Godavarthi , Johan Hovold , Doug Anderson , Stephen Boyd Subject: Re: [PATCH] Bluetooth: qca: fix device-address endianness Message-ID: References: <20231227180306.6319-1-johan+linaro@kernel.org> Precedence: bulk X-Mailing-List: linux-bluetooth@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: On Tue, Feb 13, 2024 at 05:03:15PM +0100, Johan Hovold wrote: > On Tue, Feb 13, 2024 at 03:55:06PM +0000, Matthias Kaehlcke wrote: > > On Tue, Feb 13, 2024 at 03:41:56PM +0100, Johan Hovold wrote: > > > On Wed, Dec 27, 2023 at 07:03:06PM +0100, Johan Hovold wrote: > > > > The WCN6855 firmware on the Lenovo ThinkPad X13s expects the Bluetooth > > > > device address in MSB order when setting it using the > > > > EDL_WRITE_BD_ADDR_OPCODE command. > > > > > Reverse the little-endian address before setting it to make sure that > > > > the address can be configured using tools like btmgmt or using the > > > > 'local-bd-address' devicetree property. > > > > > > > > Note that this can potentially break systems with boot firmware which > > > > has started relying on the broken behaviour and is incorrectly passing > > > > the address via devicetree in MSB order. > > > > > > > > Fixes: 5c0a1001c8be ("Bluetooth: hci_qca: Add helper to set device address") > > > > Cc: stable@vger.kernel.org # 5.1 > > > > Cc: Balakrishna Godavarthi > > > > Cc: Matthias Kaehlcke > > > > Signed-off-by: Johan Hovold > > > > > > Can we go ahead and merge this one to get this fixed in 6.8? > > > > > > I've spoken to Bjorn Andersson at Qualcomm about this and he is in > > > favour of doing so. The only people actually using the devicetree > > > property should be the Chromium team and they control their own boot > > > firmware and should be able to update it in lockstep (and Android uses > > > some custom hacks to set the address that are not in mainline). > > > > Unfortunately it's not as trivial as it sounds for Chrome OS. The boot > > firmware is controlled by Chrome OS, however for any baseboard (e.g. > > 'trogdor') there is a larger number binary firmware packages, one > > for every model derived from that baseboard. There can be dozens of > > models. Chrome OS Firmware releases are qualified and rolled out per > > model. FW qual may involve the ODM, usually there are multiple ODMs > > per board. In an absolute emergency it would be possible to coordinate > > a qual and synced rollout for all models, but it's definitely > > non-trivial in terms of operations. > > Ok, fair enough. > > Could you please provide a list of the compatible strings that you guys > currently use and I can add new compatible strings for those, while > keeping the current ones for backwards compatibility with older boot > firmware? 'qcom,wcn3991-bt' should be the only impacted compatible string for released devices. Thanks for maintaining backwards compatibility, and sorry for the inconvenience.