From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (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 F14D23B6379 for ; Sat, 15 Aug 2026 11:56:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786795008; cv=none; b=L2T7WP1HysRKZq+51zzVFyizsprsyoun3SayRnHN8vl4UmOkdC4kp04mLbwCmSXpxaTZ7kObkjzkgRicAMv01ue7N8forYygwQEOBmzlrxwWqesXq++cw3SNSgNsnDD+y48P3EfqWexGj2i71KVg/hIvcOySdUZOHDoBhB6d8Us= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786795008; c=relaxed/simple; bh=b5LQIwGVpyvdNO/WfDgWI7hUw5OYOoA0sgu5/kZeQg4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=NUdgO4dEv6ANrYA69uaS3pT8wYVLLJO3Ga9s3lVoDtnYO4ixOXiwVCzxWRakXg2S5hi5IlINB1sillEujKoW0UUpU2aaT63h14gKWIbn20exohnl4JtFVrSwBLOk4ZyneWCq9bNqv2AwQAqVxObwScRS/+rqqT2F8P4u0WKBMWs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=m4UAO/7J; arc=none smtp.client-ip=209.85.221.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="m4UAO/7J" Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-471eeac43bfso1691177f8f.3 for ; Sat, 15 Aug 2026 04:56:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786795005; x=1787399805; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=9/9+rmVvmvs6s8dnr8ttRWG5a1oyP8FaGHhemcTL1GA=; b=m4UAO/7Jrx2eKOR2GDdqDlQ1zZmhcy2bvhoZSSSY0gP4y8lHFbaAO7bCLXbcZvJRCS BupAgmNQ6zddMp2WtxJbXYgUNxOE82MwRBX1ckdJEQNiTPeawX0FiP/tg6W71Er4aiMA Po6y+G6DLG66SEmvmJ/Jd7K8gA59c0SPNCK3CRaE8AGdR0moZpfhgb54WtrgdXf8Uf7G +e0V3EDFQvsUCOCqqSzWzNJyGAlmFaX8Fmt2nWCUHa3ur2vUioUQPlDKpvFcOwyZ8QGL P2NDvJg3so7fYhd7rJDH9rrxeLH9SDvs3bug5dkxkMLv/HYPr7lsO37W06kna48R6XmP cxjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786795005; x=1787399805; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=9/9+rmVvmvs6s8dnr8ttRWG5a1oyP8FaGHhemcTL1GA=; b=aclmWeGqsrFTzFfRTUX4Lv3rt/pUEpkCh7HJWNsZmA9WoHzmFf1PMFN9q3LJrDKJzB zeK8LOe+M5NUylZRtUeAaUaST8V9B2/RzzlZklAe0sOordc9rpcumk75MfWsO0aBxOjl RYFocsoM7RocElJERcEchThLTB2VWGE/+Z47L/6zxSt2O3OFOqyi3ifnHbbDVMfXDNGp kqEe3Vooxs/Lzj8/rnU9Vc8matzQFAcUoZYE0yahz3ZQzLpEt8yFnmZ7no0rDmt/g/jw JXpoYuTOW+oE032QiiMaAmUDuB/sazGVIr1ax36rg7NOUdUjZgxtYyufhNKOLuD8YlTj nl8g== X-Forwarded-Encrypted: i=1; AHgh+RqdDR5N2LRVgmDoD9eCZFfyI21hk4N5Yxk/+7HzRfaEwxiV1y+4sdBRTIl7WY+tPy3PQR9ojv6pF8skMCM=@vger.kernel.org X-Gm-Message-State: AOJu0YxmMaMzQTDo7deX4VUoCr7I2vH6df8KcdYeBC1ABtmabR+DF6NK a5/+YlKlsmb4kq7H5/7Xq6HNGJfc9YcDSKqXkEI0oHPleqyd0YgmVCJ7 X-Gm-Gg: AR+sD10oD17lBfuu3HEpUfu3PI0j9wxnndXJMFk/puGDREGECOiuzhV8SEK/QMf/uDp u/RsUpg3VGycoD4NbmWQ+vpTLF5i+VsfArTLSvs4/wo3TvybXOcLVFk0IL0rR0GFfnHZhW9z8bA XuwFJjasQumXPCMM+qUb9aVPHGB/P0zjT1FjmRJX0FkQFSMXRxuapBuXZXMzPyg/pMNieHafV1j 22IKOzDiHCZhvHm9O4rXA9kUAKlDoUmVrXtkdziZaWLtc3W/CkyPD72hGle33LBCv7+xonsZnjM Nvn2eb1tZ58ntG6fGBo2OxHKfoVm3j82/INY90kzL5A6jTi1HZHVp9caSwjGstfEz4C1Unx7cfF rmk+1FwJ3NGi/miboCxgiKPlbODR8HR1oyQzqJPzE/Ljds0xd0mphGzhOXWoABhRR9GXFJQVF0j EmsB3iVvESYQKesoom/P9MnPGMKkVtN9fo8Z5JD7FM0xPofC3Y4ng51pEa1xUGh2bhDNkKe0h+h /yD4aKKzeTR X-Received: by 2002:a05:6000:25f5:b0:482:984d:fdd2 with SMTP id ffacd0b85a97d-482984dff83mr6257452f8f.20.1786795005196; Sat, 15 Aug 2026 04:56:45 -0700 (PDT) Received: from archlinux.tailbe3ea7.ts.net ([185.19.153.11]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4815f2c129dsm16085455f8f.26.2026.08.15.04.56.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 15 Aug 2026 04:56:44 -0700 (PDT) From: Ismail Tarim To: Marcel Holtmann , Luiz Augusto von Dentz , linux-bluetooth@vger.kernel.org Cc: Jing Cai , Sean Wang , Chris Lu , linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, Ismail Tarim Subject: [PATCH v2 2/2] Bluetooth: btmtk: Do not discard the subsystem reset timeout Date: Sat, 15 Aug 2026 14:56:24 +0300 Message-ID: <20260815115624.8309-3-ismailtarim7@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260815115624.8309-1-ismailtarim7@gmail.com> References: <20260815115624.8309-1-ismailtarim7@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit When the MTK_BT_RST_DONE poll times out, btmtk_usb_subsys_reset() logs "Reset timeout" and keeps the error in err, but err is then overwritten by the return value of the following btmtk_usb_id_get() call, so the timeout is never reported to the caller. Commit 25b6d7593a3a ("Bluetooth: btmtk: introduce btmtk reset work") discarded the return value of the chip id read, so the function returned the timeout error as intended. Commit 3dcb122b3064 ("Bluetooth: btusb: mediatek: return error for failed reg access") started assigning err at that call and silently dropped it. Keep the timeout in a separate variable and return it, restoring the original behaviour without changing the control flow. Fixes: 3dcb122b3064 ("Bluetooth: btusb: mediatek: return error for failed reg access") Signed-off-by: Ismail Tarim --- drivers/bluetooth/btmtk.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/bluetooth/btmtk.c b/drivers/bluetooth/btmtk.c index dc702c0a6034..c0ed51567ed4 100644 --- a/drivers/bluetooth/btmtk.c +++ b/drivers/bluetooth/btmtk.c @@ -860,6 +860,7 @@ static u32 btmtk_usb_reset_done(struct hci_dev *hdev) int btmtk_usb_subsys_reset(struct hci_dev *hdev, u32 dev_id) { + int reset_err = 0; u32 val; int err; @@ -958,8 +959,10 @@ int btmtk_usb_subsys_reset(struct hci_dev *hdev, u32 dev_id) err = readx_poll_timeout(btmtk_usb_reset_done, hdev, val, val & MTK_BT_RST_DONE, 20000, 1000000); - if (err < 0) + if (err < 0) { bt_dev_err(hdev, "Reset timeout"); + reset_err = err; + } if (dev_id == 0x7922) { err = btmtk_usb_uhw_reg_write(hdev, MTK_UDMA_INT_STA_BT, 0x000000FF); @@ -973,7 +976,7 @@ int btmtk_usb_subsys_reset(struct hci_dev *hdev, u32 dev_id) return err ? err : -ENODEV; } - return err; + return reset_err; } EXPORT_SYMBOL_GPL(btmtk_usb_subsys_reset); -- 2.55.0