From nobody Sun Feb 8 07:21:34 2026 Received: from mail-oa1-f41.google.com (mail-oa1-f41.google.com [209.85.160.41]) (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 BD63C25E828 for ; Wed, 26 Feb 2025 22:54:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740610492; cv=none; b=eJBii1om8Kak6rXANmWHIhwPt7riyv7k+JllQqpJ9w0H7IZ7oCNzEFUsdFayUW/pTOLpgm1UbZUxWdgXV1KsC4Hq1WgYbtS072jq1DXQcK4e55kMvxLHd/WxZPfvUz4rwt918LjoHO2Wygnq4VAVFSYMUcRtjqEwylzkokO+CIM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740610492; c=relaxed/simple; bh=bD0MLklLJmd4xonr6bt0CWzIrOHzAQKh1+43nLJDmBc=; h=From:To:Cc:Subject:Date:Message-Id; b=WX+qEsQmNMKzehDWjF8vmZTpqEpzVYBR8aMVgm/6L8CbsyQuYd5XOm7eh8cxnoB3PVX58E+frD0OoyB1H00bBJtWA3S8Hwu21z/PTpwlcOax94Aaap62HSniwfOWtvfhEGFo6BBw2mOQ6p1s1AspWhvu24tphZ62X271fkde0So= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=iPhkPEQW; arc=none smtp.client-ip=209.85.160.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="iPhkPEQW" Received: by mail-oa1-f41.google.com with SMTP id 586e51a60fabf-2adc2b6837eso128904fac.1 for ; Wed, 26 Feb 2025 14:54:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1740610490; x=1741215290; darn=vger.kernel.org; h=message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=MPNaIL+tlppk9dTrhKd24jiwzkfnb9qsPkVFFQkEVaM=; b=iPhkPEQWBcRQkiM8lzzo14uOFex4XNjOXiFBdZXN7vpPSTNSRItaZES6Cr0P4xxXO0 RguBCsIhEHJMoDJGEz8Xhu+oCXcHJJqt7ickvYnshcAJbp25oWnYaH/Q/Hvg0bSIqSxv EpRK9n1iEjosBP9JWofsCdlVLQ08bXQLE1CBo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740610490; x=1741215290; h=message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=MPNaIL+tlppk9dTrhKd24jiwzkfnb9qsPkVFFQkEVaM=; b=J3z+HwP9+cxgk0vuUDIHMxnVP8r8Y/RAt6tpyJhv7q3QQJPppIgbgRGogNMgN24TBW u49wBkqvXw7WqXNdt/FDEXC9MTTIGRMoglYTCZCbETVuYJbPa2hkkxBZAmtWYz7A/DkX QK/ukQNQGTh/IhUV5ETGiSKVcpVuJclbqIvivj/NCVUVB3/7T5SRsGuG/UKPISXDZ9l8 vj4AnZzIVkAfKcwOmJtrZIf9NFpi+hLWYoS5iJxsgG6rgOiFTiyn2+juh97lDbDjDamD 3BCyO8wowwajjMiocQ1ewod4sEgE7DvvYUEQzFKuxp5pyXwfft1vx2z0aXGBTD2q0ufU QErA== X-Forwarded-Encrypted: i=1; AJvYcCXfkFeP+7KDvRK4B/jaUjInMWeOh6DqB6R8QfyNnI6te09LUgLbgeDPa/YwB8FaOLZs/QQF7j/bfh7X65g=@vger.kernel.org X-Gm-Message-State: AOJu0Yx2ji5weQGLNR+pZGamab/Z32fWazGlT1G9KRJHg+1LLXSOVX9h YEoaqAg1wxoMtTjnYfOx9QGh++JuHyp0w7Mr8YQunnyCMrlKhOveuCdjM3NPNA== X-Gm-Gg: ASbGncuEHEWSAPJWixyqiz0brE+GFgp98DqYEJx/egs7IKh7IP+xdfp6kXhphNd8trT gk2Prsq+TXmYTk988e0MM8FB5qoiCIKv4F3EEK3DAU2Z66Qv/Y7E7ON/Wo6Ie8wlyXpUuzF0qkw Lw2x6K2Ja4dOMnW4e3TjxG2AuIq9NLWhWxpKFoQEApXuP6D0qUbMbZw2WE8EdiKiWyEuws2MQD+ jFXguzrLA5YCA5sIqFK+Qffq1rQe9KKoFgvkP3EPgbrZm9wNerwq9QmmVXFrVJ+vB7Kbl+uyRSS TUoyLVHvCpRteQR3cmLrsZLAWIPnp1H1b/Jn5EyfW0Wn2XcdOfaanSCxetEgRA== X-Google-Smtp-Source: AGHT+IHUIW/ZU2Dggyf5u1+gjtFIIkhmGjLUrnOsGskPpBQmRzW6dpzyhd88y6Pq0rxrIi1uxGS7YA== X-Received: by 2002:a05:6870:9e0d:b0:27d:10f5:347 with SMTP id 586e51a60fabf-2c10f2795e7mr4704632fac.15.1740610489717; Wed, 26 Feb 2025 14:54:49 -0800 (PST) Received: from mail.broadcom.net ([192.19.144.250]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-728afd764f7sm33150a34.47.2025.02.26.14.54.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 26 Feb 2025 14:54:49 -0800 (PST) From: Kamal Dasu To: florian.fainelli@broadcom.com, Brian Norris , Kamal Dasu , Broadcom internal kernel review list , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Cc: Kamal Dasu Subject: [PATCH] mtd: rawnand: brcmnand: fix PM resume warning Date: Wed, 26 Feb 2025 17:53:09 -0500 Message-Id: <20250226225356.16380-1-kamal.dasu@broadcom.com> X-Mailer: git-send-email 2.17.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Fixed warning due to unintialized struct nand_operation op->cs WARN_ON(op->cs >=3D nanddev_ntargets(&chip->base) [ 14.588522] ------------[ cut here ]------------ [ 14.588529] WARNING: CPU: 0 PID: 1392 at drivers/mtd/nand/raw/internals.= h:139 nand_reset_op+0x1e0/0x1f8 [ 14.588553] Modules linked in: bdc udc_core [ 14.588579] CPU: 0 UID: 0 PID: 1392 Comm: rtcwake Tainted: G W = 6.14.0-rc4-g5394eea10651 #16 [ 14.588590] Tainted: [W]=3DWARN [ 14.588593] Hardware name: Broadcom STB (Flattened Device Tree) [ 14.588598] Call trace: [ 14.588604] dump_backtrace from show_stack+0x18/0x1c [ 14.588622] r7:00000009 r6:0000008b r5:60000153 r4:c0fa558c [ 14.588625] show_stack from dump_stack_lvl+0x70/0x7c [ 14.588639] dump_stack_lvl from dump_stack+0x18/0x1c [ 14.588653] r5:c08d40b0 r4:c1003cb0 [ 14.588656] dump_stack from __warn+0x84/0xe4 [ 14.588668] __warn from warn_slowpath_fmt+0x18c/0x194 [ 14.588678] r7:c08d40b0 r6:c1003cb0 r5:00000000 r4:00000000 [ 14.588681] warn_slowpath_fmt from nand_reset_op+0x1e0/0x1f8 [ 14.588695] r8:70c40dff r7:89705f41 r6:36b4a597 r5:c26c9444 r4:c26b0048 [ 14.588697] nand_reset_op from brcmnand_resume+0x13c/0x150 [ 14.588714] r9:00000000 r8:00000000 r7:c24f8010 r6:c228a3f8 r5:c26c94bc= r4:c26b0040 [ 14.588717] brcmnand_resume from platform_pm_resume+0x34/0x54 [ 14.588735] r5:00000010 r4:c0840a50 [ 14.588738] platform_pm_resume from dpm_run_callback+0x5c/0x14c [ 14.588757] dpm_run_callback from device_resume+0xc0/0x324 [ 14.588776] r9:c24f8054 r8:c24f80a0 r7:00000000 r6:00000000 r5:00000010= r4:c24f8010 [ 14.588779] device_resume from dpm_resume+0x130/0x160 [ 14.588799] r9:c22539e4 r8:00000010 r7:c22bebb0 r6:c24f8010 r5:c22539dc= r4:c22539b0 [ 14.588802] dpm_resume from dpm_resume_end+0x14/0x20 [ 14.588822] r10:c2204e40 r9:00000000 r8:c228a3fc r7:00000000 r6:0000000= 3 r5:c228a414 [ 14.588826] r4:00000010 [ 14.588828] dpm_resume_end from suspend_devices_and_enter+0x274/0x6f8 [ 14.588848] r5:c228a414 r4:00000000 [ 14.588851] suspend_devices_and_enter from pm_suspend+0x228/0x2bc [ 14.588868] r10:c3502910 r9:c3501f40 r8:00000004 r7:c228a438 r6:c0f95e1= 8 r5:00000000 [ 14.588871] r4:00000003 [ 14.588874] pm_suspend from state_store+0x74/0xd0 [ 14.588889] r7:c228a438 r6:c0f934c8 r5:00000003 r4:00000003 [ 14.588892] state_store from kobj_attr_store+0x1c/0x28 [ 14.588913] r9:00000000 r8:00000000 r7:f09f9f08 r6:00000004 r5:c3502900= r4:c0283250 [ 14.588916] kobj_attr_store from sysfs_kf_write+0x40/0x4c [ 14.588936] r5:c3502900 r4:c0d92a48 [ 14.588939] sysfs_kf_write from kernfs_fop_write_iter+0x104/0x1f0 [ 14.588956] r5:c3502900 r4:c3501f40 [ 14.588960] kernfs_fop_write_iter from vfs_write+0x250/0x420 [ 14.588980] r10:c0e14b48 r9:00000000 r8:c25f5780 r7:00443398 r6:f09f9f6= 8 r5:c34f7f00 [ 14.588983] r4:c042a88c [ 14.588987] vfs_write from ksys_write+0x74/0xe4 [ 14.589005] r10:00000004 r9:c25f5780 r8:c02002f0 r7:00000000 r6:0000000= 0 r5:c34f7f00 [ 14.589008] r4:c34f7f00 [ 14.589011] ksys_write from sys_write+0x10/0x14 [ 14.589029] r7:00000004 r6:004421c0 r5:00443398 r4:00000004 [ 14.589032] sys_write from ret_fast_syscall+0x0/0x5c [ 14.589044] Exception stack(0xf09f9fa8 to 0xf09f9ff0) [ 14.589050] 9fa0: 00000004 00443398 00000004 00443398 = 00000004 00000001 [ 14.589056] 9fc0: 00000004 00443398 004421c0 00000004 b6ecbd58 00000008 = bebfbc38 0043eb78 [ 14.589062] 9fe0: 00440eb0 bebfbaf8 b6de18a0 b6e579e8 [ 14.589065] ---[ end trace 0000000000000000 ]--- The fix uses the higher level nand_reset(chip, chipnr); where chipnr =3D 0,= when doing PM resume operation in compliance with the controller support for sin= gle die nand chip. This ensures that the op->cs is initialized before calling the nand_reset_op(). Fixes : 97d90da8a886 ("mtd: nand: provide several helpers to do common NAND= operations") Signed-off-by: Kamal Dasu --- drivers/mtd/nand/raw/brcmnand/brcmnand.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c b/drivers/mtd/nand/ra= w/brcmnand/brcmnand.c index fea5b6119956..17f6d9723df9 100644 --- a/drivers/mtd/nand/raw/brcmnand/brcmnand.c +++ b/drivers/mtd/nand/raw/brcmnand/brcmnand.c @@ -3008,7 +3008,7 @@ static int brcmnand_resume(struct device *dev) brcmnand_save_restore_cs_config(host, 1); =20 /* Reset the chip, required by some chips after power-up */ - nand_reset_op(chip); + nand_reset(chip, 0); } =20 return 0; --=20 2.17.1