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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 70DD0C433EF for ; Mon, 4 Oct 2021 17:46:55 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 3B3D861131 for ; Mon, 4 Oct 2021 17:46:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 3B3D861131 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:In-Reply-To:MIME-Version:Date: Message-ID:From:References:Cc:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=7vHZGhqOHJ0hFChUv8HEJPIPryD/2jzmLEUu9TUhaJE=; b=2DtuDuxky3xU1Am2rd2xeCn+NF uH45DHYGWAApjO7X1YNYu7oKQSNgjyH9+dRlNv2KZASDgR2s/hcVVmlyZ4OHCA4+sLh20TBDteiBt 1xDK3NoBIOyTK35jbN0DFKh5ZxOYGfcL40u17Uep8xjEP2Lzs71n5CE3SBP1ErsCmbYK8N499Fb4i k73qq6GLDDDJl18Yz0yUJMLE0JDV65bfC361TpK7wtg5D1799ou+c6vgG1pShJ+BNZA005lQcUTa7 gAgStcN0GGg7+WnwGzrZEXE/ASvGBumlB5jxXNvmsrPZeK/BwyMHbNT93qap9VC/WptLX3J9MzsR+ TG2BkHww==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXS13-007QHV-SY; Mon, 04 Oct 2021 17:44:42 +0000 Received: from mail-pl1-x632.google.com ([2607:f8b0:4864:20::632]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXS10-007QGq-Qu; Mon, 04 Oct 2021 17:44:40 +0000 Received: by mail-pl1-x632.google.com with SMTP id x4so422331pln.5; Mon, 04 Oct 2021 10:44:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=2v3glRkX+dSyZeU+LpE78smFocLlU92ly5f6PxfpGIk=; b=exM0K+E4yB1Gke5jtGF93uVyP5I0tk4gMW4/RWH7QH4IyIJc1O3pfnKAxDKzuZ7JET Nmvk37dsNFWz4Gma8j3wONGUd26+3pofxU7if1Pf5JnTDou2hO/xkoCf1rue4ZlpZTul 99xCBJRJgmuQUx4onXyXmWgZ8Lm22WzO03a+DfiJSVuOdckTBB/r6iVBjSU9eyKg+cRm t/ecyflTAzFbIoUQLxvkFMy0CrCsg1gVg99ygcEOWdjYdvq5K3bmFJYN/RE6n4s7ios7 BGofyI7hNUoJfHZmeu5acx+whFmHrwYzm4/uH+/cpviGT56ak5nK3bqm6atd+SBnwAH9 3l2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=2v3glRkX+dSyZeU+LpE78smFocLlU92ly5f6PxfpGIk=; b=lmtSRqvcm5ezmwJmanIi9mxZwKmnpEKjmrIT/QVqmKHhZh1p3A9ci/4RArtYWnuj/O ZN36/czpQdIhZyTjojUAMPbyui85G6uK2w/ubt7Y5ajYJLwsfZFSydMUjBR8dt5i6OVW +zG1JdhQXVMG7DBUI+3GOK4do6Pnh4saANOWTqljKmWyPPhRE9gQdfVeIcL+8Uoyl8R9 wnX0oC3NRNoHb+r9E3P8JWm0NE0j8ix9GJ4Xw02bBzV7W3pPhxEDJcf0EMvdwEC4+qQB P1UMktviRy7SLFHv0gVRaiuzREkU6oRpBExU1aHlXBqpq4mCRxWXnDFJkwnwMOFCwQ34 /fmg== X-Gm-Message-State: AOAM533R0ramqNSWh4FwVGYAiGcfg5LNgWaYzRontpBmFAb0iwL8aGnW 6K4du6c/TfxCaTE+4QoNUGU= X-Google-Smtp-Source: ABdhPJxVsJeHJjtVjulyj7Lvd3dU/a9yFbWHkKKERRqOsy+WcmiF5G5R+5VTY+dk8Jd7jIdkhv4R3w== X-Received: by 2002:a17:90a:4812:: with SMTP id a18mr37992614pjh.40.1633369477657; Mon, 04 Oct 2021 10:44:37 -0700 (PDT) Received: from [10.67.48.245] ([192.19.223.252]) by smtp.googlemail.com with ESMTPSA id p2sm13649148pja.51.2021.10.04.10.44.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 04 Oct 2021 10:44:37 -0700 (PDT) Subject: Re: [PATCH] spi: bcm2835: do not unregister controller in shutdown handler To: Mark Brown , Jason Gunthorpe Cc: Lino Sanfilippo , rjui@broadcom.com, sbranden@broadcom.com, bcm-kernel-feedback-list@broadcom.com, nsaenz@kernel.org, linux-spi@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, p.rosenberger@kunbus.com, linux-integrity@vger.kernel.org, stable@vger.kernel.org References: <2c4d7115-7a02-f79e-c91b-3c2dd54051b2@gmx.de> <20211004131756.GW3544071@ziepe.ca> <20211004154436.GY3544071@ziepe.ca> <20211004165127.GZ3544071@ziepe.ca> <20211004171301.GA3544071@ziepe.ca> From: Florian Fainelli Message-ID: <8513334a-1de4-bc9c-0157-e792e8ff4871@gmail.com> Date: Mon, 4 Oct 2021 10:44:34 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211004_104438_905848_FEB3B799 X-CRM114-Status: GOOD ( 14.12 ) 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 On 10/4/21 10:27 AM, Mark Brown wrote: > On Mon, Oct 04, 2021 at 02:13:01PM -0300, Jason Gunthorpe wrote: > >> I'm kind of surprised a scheme like this didn't involve a FW call >> after Linux is done with the CPUs to quiet all the HW and let it >> sleep, I've built things that way before at least. > > That's a *lot* of code to put in firmware if you can't physically power > most of the system down. Indeed, and that also assume it may be possible for firmware to have the last say, which is not necessarily possible (though that ought to be a system design issue that would need fixing). It seems reasonable to me to delegate the powering off of the hardware to the respective Linux drivers since they ought to be in the best position to make appropriate decisions for the hardware they control. Anyway, we are divergin slightly here, how do we go about fixing .shutdown here? -- Florian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel