Received: by 2002:a25:683:0:0:0:0:0 with SMTP id 125csp1356716ybg; Tue, 2 Jun 2020 07:54:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxTHmuJiuntc0+/IE2uT6A131x7tYK7Y03ZQIfd5GcGtO17J01dzLQlsVY3L4raZVH4Jbbs X-Received: by 2002:a17:906:768e:: with SMTP id o14mr3066228ejm.497.1591109645823; Tue, 02 Jun 2020 07:54:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1591109645; cv=none; d=google.com; s=arc-20160816; b=IXk+kOqc8eY96qv7Bk7szGwyL45AVO9w5XADV4zrzi5/hvavAbFVvcxXk2aNqQ6NGq +HGf7i/xAN7U6XTgpdw31n66nkQfJdiywAEbRKuDESG/YDFkeDTYz+8okmriHZ6G0F5b YOw5ZD1rbNXiNoTUX82qNh1umo+sVMI6PcLx9scbHJ42+ZeBWVqxpOpe7WwCmoI3zzVq fZBN0Aa59v9nY0ROCJiZpkQ5+DWzLYIKiY9QXGFB0nX4Q3DSedTyH6vWqnB4JXrUPynl jGGvi7eukpQkf9yVlrmpW6wAeLt8KjEvYyQFEqDvlc/0o27QoOIeoATMw2cEj5sKNU7y EvBw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:message-id:references :in-reply-to:subject:cc:to:from:date:content-transfer-encoding :mime-version:dkim-signature; bh=rlURGUY597TqbB43gaR7bgIPpydKPkFEMWK7XxPMQgc=; b=qWkCJ0EuoY0yQ8N0ktSUnEF9JEj75xoDCg5T4XMPvLz6pOL/yUSl3VXVpQDvL0VgV0 iIz3pwjIIKkMyBwNO3E30s3NdKktc16nkb+V5wxChXcZP498uVqqQrCJUMaJ4z8eSYUa Zfo9iELIC75RRu4V71KZ/83R43GBe/czMWzWE+vBlFimMC/SDnDXH6b6HpP7aRkWxa9e WwihFOaRSYHSEaucq4Y6kBBbbE81g1rcoGmMxbxevEk/84ZViPdajm15vH8iU7LR1OZ5 AjhNb977vapyR2eBh9EI0+HbxeiPFHvwNPMOZgqHQWlsS3jy0gnoTzq3c4LPhSKrr08q SjCw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@mg.codeaurora.org header.s=smtp header.b=RXa143qu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dc20si1576887edb.75.2020.06.02.07.53.41; Tue, 02 Jun 2020 07:54:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@mg.codeaurora.org header.s=smtp header.b=RXa143qu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726461AbgFBOto (ORCPT + 99 others); Tue, 2 Jun 2020 10:49:44 -0400 Received: from mail27.static.mailgun.info ([104.130.122.27]:62612 "EHLO mail27.static.mailgun.info" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726373AbgFBOtn (ORCPT ); Tue, 2 Jun 2020 10:49:43 -0400 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1591109383; h=Message-ID: References: In-Reply-To: Subject: Cc: To: From: Date: Content-Transfer-Encoding: Content-Type: MIME-Version: Sender; bh=rlURGUY597TqbB43gaR7bgIPpydKPkFEMWK7XxPMQgc=; b=RXa143quTDR7CUXFT7p2ov+UPqxNHknahyAhCAW21Sv7ZvOXjTe41E9J7/9CdQmLjBqy4kVk u3IXuXCmpKYahb7DI3eNDbQeIaodyLUH1dY4gmOmtMQUA/u5bRIWp1/eeF4XrAR/zppWp1yw E40QQw6ZLdVs1knA+b/Vpe9AdL8= X-Mailgun-Sending-Ip: 104.130.122.27 X-Mailgun-Sid: WyI0MWYwYSIsICJsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnIiwgImJlOWU0YSJd Received: from smtp.codeaurora.org (ec2-35-166-182-171.us-west-2.compute.amazonaws.com [35.166.182.171]) by smtp-out-n04.prod.us-east-1.postgun.com with SMTP id 5ed666fe2c54998475e15258 (version=TLS1.2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256); Tue, 02 Jun 2020 14:49:34 GMT Received: by smtp.codeaurora.org (Postfix, from userid 1001) id 10E51C433CB; Tue, 2 Jun 2020 14:49:34 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-caf-mail-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=2.0 tests=ALL_TRUSTED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.codeaurora.org (localhost.localdomain [127.0.0.1]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: saiprakash.ranjan) by smtp.codeaurora.org (Postfix) with ESMTPSA id 812F4C433C9; Tue, 2 Jun 2020 14:49:33 +0000 (UTC) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Tue, 02 Jun 2020 20:19:33 +0530 From: Sai Prakash Ranjan To: Emil Velikov Cc: Krishna Manikandan , ML dri-devel , linux-arm-msm , freedreno@lists.freedesktop.org, devicetree , "Linux-Kernel@Vger. Kernel. Org" , Sean Paul , kalyan_t@codeaurora.org, "Kristian H . Kristensen" , mka@chromium.org, devicetree-owner@vger.kernel.org Subject: Re: [v2] drm/msm: add shutdown support for display platform_driver In-Reply-To: References: <1591009402-681-1-git-send-email-mkrishn@codeaurora.org> Message-ID: X-Sender: saiprakash.ranjan@codeaurora.org User-Agent: Roundcube Webmail/1.3.9 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Emil, On 2020-06-02 19:43, Emil Velikov wrote: > Hi Krishna, > > On Tue, 2 Jun 2020 at 08:17, Krishna Manikandan > wrote: >> >> Define shutdown callback for display drm driver, >> so as to disable all the CRTCS when shutdown >> notification is received by the driver. >> >> This change will turn off the timing engine so >> that no display transactions are requested >> while mmu translations are getting disabled >> during reboot sequence. >> >> Signed-off-by: Krishna Manikandan >> > AFAICT atomics is setup in msm_drm_ops::bind and shutdown in > msm_drm_ops::unbind. > > Are you saying that unbind never triggers? If so, then we should > really fix that instead, since this patch seems more like a > workaround. > Which path do you suppose that the unbind should be called from, remove callback? Here we are talking about the drivers which are builtin, where remove callbacks are not called from the driver core during reboot/shutdown, instead shutdown callbacks are called which needs to be defined in order to trigger unbind. So AFAICS there is nothing to be fixed. msm_pdev_shutdown() platform_drv_shutdown() device_shutdown() kernel_restart_prepare() kernel_restart() __arm64_sys_reboot() Thanks, Sai -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation