From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7C4E2C433F5 for ; Tue, 15 Feb 2022 20:35:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=DHgT/HY+A3Gyc1OgbbdptS1AKDwfHP37S7Ym8M+FiNE=; b=x1d51XTfzcyrOQ 2qYZFh9g72waQZQaTXs5tJA9e2CkbF+3UOfU8ojYW+1KFUJaJmWm8Fmr4P02vyGJurtQfad4XcwYq WIkGqzo3Se+rsMpz8a5ptLwUib3k+/Fswi6iDCzRUSf/g5ZYjZJcFj709pYzNFQje+nnMZzuQ19eh v2pPYMH0WcbY+PHzPbSU8LYjl+j/bN1rK54fxTEz6pfU025heOye8K5NXWWhYXlDNlFBwLwSnbs14 okbO7kXcX1SCH+sLrxyn4X92Owz52R56eHPxBQdcBILEIUyngZlUmTW9SEMdRnK6IANdtb6wveWbJ y0sZ3IgSATLmUluOd0Og==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nK4X6-004Z4z-HH; Tue, 15 Feb 2022 20:34:44 +0000 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nK4X2-004Z48-Dt for linux-arm-kernel@lists.infradead.org; Tue, 15 Feb 2022 20:34:41 +0000 Received: by mail-wr1-x436.google.com with SMTP id d27so15199wrc.6 for ; Tue, 15 Feb 2022 12:34:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=KRSkNhIEhZMEMhWXs5iKQNZLmkUM8XOFNeLt27M+yH0=; b=hmXooVo0/3oV8QfjJUBlso0REBtT/7qZxc4FtjAmra0e1picrf+m3AyZe/9+8D/Qys dAKgFh2OJUrLVKJkf6aJmpRcU7Qan23dlXdC0svDE/UgTCkjX2BuMtea43ycfowTGPAy UI++sJoOrZY7O3tZRi5Sd98UNVeeD3+7UCSQRtdd8UB27UASvbssCSCmBM7W9NCdoRSy EwGs+mFo+Q+4AYFJcptcxtmga80wOuzuZaV1sd5mPdtoreJ+MuJe34KIj6aYUmlcWdX7 ZPeKhgH4e1WwI7lmO1AO+RdrMWmX7QTBeP2Eg6Q5AY0fo0HHAZP//R0Ljxtl8OJ3GqRG eDWQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=KRSkNhIEhZMEMhWXs5iKQNZLmkUM8XOFNeLt27M+yH0=; b=7HpMtdXR/g5NcUPEpjUfSmrBLdkTyRjXn5XpkImnd+EtkHAiexgQ8zeCVsq0o0MZh0 vcVlVbFclv8rkRxImlASMEpFW7iRPHDTz+xCwJjB9fQhTxT7kNhXULAsITkr4e+pCtza rSDSDXBXGjz8+x9rzuuw2cVt5tL9OAtu4u58XiL2kqIvw7kgLkm8hP+2tNyqfjl/S4hq Sl40nmQT2eSyVsyrTSMBIgYxW6l+JhWucXMQJLnZk+w6yadid5LdKMEhHCPIov1jGAF4 r4+sc9ASXkhJSO8FBmBoDzcO2uqgHX0lN4dFUou4yjxp0Mf1xTbBOlpF4DPKmFEAOiPI xuXA== X-Gm-Message-State: AOAM532LktmscvauWq9LeqpXxmb1c4mZS8uLnMZoq3ZyUXPvje5fEDQJ geKd+zgQ0DMdyggwA+Vs6ZE= X-Google-Smtp-Source: ABdhPJx7LPF22AwP5kqzNKnc1tpYmsQB+V9uLmcnJHL/G901Xck085/9xsudMsuFVZQmfQc0vJM0Ug== X-Received: by 2002:adf:f191:: with SMTP id h17mr568730wro.188.1644957277766; Tue, 15 Feb 2022 12:34:37 -0800 (PST) Received: from kista.localnet (cpe-86-58-32-107.static.triera.net. [86.58.32.107]) by smtp.gmail.com with ESMTPSA id r14sm6034914wrz.84.2022.02.15.12.34.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 15 Feb 2022 12:34:37 -0800 (PST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Guenter Roeck Cc: mripard@kernel.org, wens@csie.org, robh+dt@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Klein , Hans Verkuil Subject: Re: Re: [PATCH v2] ARM: dts: sun8i: Adjust power key nodes Date: Tue, 15 Feb 2022 21:34:36 +0100 Message-ID: <4714494.31r3eYUQgx@kista> In-Reply-To: <20220215002732.GA3215504@roeck-us.net> References: <20211129165510.370717-1-jernej.skrabec@gmail.com> <20220215002732.GA3215504@roeck-us.net> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220215_123440_505243_0A4C8D86 X-CRM114-Status: GOOD ( 20.68 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi! Dne torek, 15. februar 2022 ob 01:27:32 CET je Guenter Roeck napisal(a): > Hi, > > On Mon, Nov 29, 2021 at 05:55:10PM +0100, Jernej Skrabec wrote: > > Several H3 and one H2+ board have power key nodes, which are slightly > > off. Some are missing wakeup-source property and some have BTN_0 code > > assigned instead of KEY_POWER. > > > > Adjust them, so they can function as intended by designer. > > > > Co-developed-by: Michael Klein > > Signed-off-by: Michael Klein > > Signed-off-by: Jernej Skrabec > > This patch results in the following traceback when rebooting an > orangepi-pc qemu emulation. > > [ 30.899594] > [ 30.899685] ============================================ > [ 30.899757] WARNING: possible recursive locking detected > [ 30.899938] 5.17.0-rc3-00394-gc849047c2473 #1 Not tainted > [ 30.900055] -------------------------------------------- > [ 30.900124] init/307 is trying to acquire lock: > [ 30.900246] c2dfe27c (&irq_desc_lock_class){-.-.}-{2:2}, at: __irq_get_desc_lock+0x58/0xa0 > [ 30.900900] > [ 30.900900] but task is already holding lock: > [ 30.900974] c3c0ac7c (&irq_desc_lock_class){-.-.}-{2:2}, at: __irq_get_desc_lock+0x58/0xa0 > [ 30.901101] > [ 30.901101] other info that might help us debug this: > [ 30.901188] Possible unsafe locking scenario: > [ 30.901188] > [ 30.901262] CPU0 > [ 30.901301] ---- > [ 30.901339] lock(&irq_desc_lock_class); > [ 30.901411] lock(&irq_desc_lock_class); > [ 30.901480] > [ 30.901480] *** DEADLOCK *** > [ 30.901480] > [ 30.901554] May be due to missing lock nesting notation > [ 30.901554] > [ 30.901657] 4 locks held by init/307: > [ 30.901724] #0: c1f29f18 (system_transition_mutex){+.+.}-{3:3}, at: __do_sys_reboot+0x90/0x23c > [ 30.901889] #1: c20f7760 (&dev->mutex){....}-{3:3}, at: device_shutdown+0xf4/0x224 > [ 30.902016] #2: c2e804d8 (&dev->mutex){....}-{3:3}, at: device_shutdown+0x104/0x224 > [ 30.902138] #3: c3c0ac7c (&irq_desc_lock_class){-.-.}-{2:2}, at: __irq_get_desc_lock+0x58/0xa0 > [ 30.902281] > [ 30.902281] stack backtrace: > [ 30.902462] CPU: 0 PID: 307 Comm: init Not tainted 5.17.0-rc3-00394- gc849047c2473 #1 > [ 30.902572] Hardware name: Allwinner sun8i Family > [ 30.902781] unwind_backtrace from show_stack+0x10/0x14 > [ 30.902895] show_stack from dump_stack_lvl+0x68/0x90 > [ 30.902970] dump_stack_lvl from __lock_acquire+0x1680/0x31a0 > [ 30.903047] __lock_acquire from lock_acquire+0x148/0x3dc > [ 30.903118] lock_acquire from _raw_spin_lock_irqsave+0x50/0x6c > [ 30.903197] _raw_spin_lock_irqsave from __irq_get_desc_lock+0x58/0xa0 > [ 30.903282] __irq_get_desc_lock from irq_set_irq_wake+0x2c/0x19c > [ 30.903366] irq_set_irq_wake from irq_set_irq_wake+0x13c/0x19c > [ 30.903442] irq_set_irq_wake from gpio_keys_suspend+0x80/0x1a4 > [ 30.903523] gpio_keys_suspend from gpio_keys_shutdown+0x10/0x2c > [ 30.903603] gpio_keys_shutdown from device_shutdown+0x180/0x224 > [ 30.903685] device_shutdown from __do_sys_reboot+0x134/0x23c > [ 30.903764] __do_sys_reboot from ret_fast_syscall+0x0/0x1c > [ 30.903894] Exception stack(0xc584ffa8 to 0xc584fff0) > [ 30.904013] ffa0: 01234567 000c623f fee1dead 28121969 01234567 00000000 > [ 30.904117] ffc0: 01234567 000c623f 00000001 00000058 000d85c0 00000000 00000000 00000000 > [ 30.904213] ffe0: 000d8298 be84ddf4 000918bc b6eb0edc > [ 30.905189] reboot: Restarting system > > The warning is no longer seen after reverting this patch. > > The problem exists but is not seen in v5.17-rc4 because a bug in commit > 8df89a7cbc63 ("pinctrl-sunxi: don't call pinctrl_gpio_direction()") > hides it. That problem is fixed with commit 3c5412cdec9f ("pinctrl-sunxi: > sunxi_pinctrl_gpio_direction_in/output: use correct offset") in linux-next, > and the traceback is seen there. Hm... These DT changes were tested with many users on older kernels for some time now and new properties conform to bindings. Should we revert pinctrl changes? Best regards, Jernej _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel