Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp7022730ybi; Thu, 13 Jun 2019 08:17:35 -0700 (PDT) X-Google-Smtp-Source: APXvYqyoPlyskA9vsFQYRqT8Ku4mfa5Db/lkZd3dZWqGTJ4rYd0ZTqZwoWYkYwPRY2lk0pCMGcxs X-Received: by 2002:aa7:808a:: with SMTP id v10mr2546197pff.170.1560439055868; Thu, 13 Jun 2019 08:17:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1560439055; cv=none; d=google.com; s=arc-20160816; b=XXHMqJFBXrheZ5EpHo3tCF3ywRtOEI+pjMD8GkYWsbmMkJHiy+Q1lx12cvHnVMlO4u //zCGRexA5cjzAf7CkDVurUi3YWBm0k3+BLFkS/bYzl632o9tSA37gbEV2DrI6Vik5/R XuWWlvUhLy39nN4D60zNajRgPNKbBhT+zD6OkuQ0xoqNFoCvPRISs1Bd/uArLhwRAwi0 KmP49Om7skjNbfBy4pwvIU8I4v9+iIllXt5Jng+eOfl9oozNDhA1WnMpchfn97FM/7UO PnLoOdE/2ZIfpP2d27SJlrYDQ0m2grE4LHBztdQFL6R1VPlAY1yAr2xhZ3xF4vzdj9xh jy2A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=jPOYrmaleIUegcZpCfbHsM+zNs7jBU9MJoJrt2dE2wk=; b=KGYyTSjLDKJwc+w+fzvEh25Xl0o8KisNzW+rWvMDQOcf51qRCKgmwM98gbfFP/pdqA 8Ai5W3Lp18gY38qtOJKVjITzlNjSLjYE9SjBn+cIrnDBDoPoJFumUFOnVS2KWHQ/HcB5 VmkaoFdBGIkRv8FoHtIW1ccWRCStnNhV5mOttY2uwZ2/o7Ajd84xiQpLbn3MvTKeG8oK Al9rr2lwNxlzPUP8M/lbHtcN6Fx7lE6zA53rwkjlzqg4mumcQlw4Fj5990yMtQr7eAZl s9E250Lf4uHHlEn1yp5gCgYuUP2ZHJdgcbsgIVh2n8Orp0qnN4aVuPHZ+RM3SBiJTNXp hm/w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 74si39139pga.291.2019.06.13.08.17.20; Thu, 13 Jun 2019 08:17:35 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732294AbfFMPPJ (ORCPT + 99 others); Thu, 13 Jun 2019 11:15:09 -0400 Received: from relay2-d.mail.gandi.net ([217.70.183.194]:44047 "EHLO relay2-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732233AbfFMN0K (ORCPT ); Thu, 13 Jun 2019 09:26:10 -0400 X-Originating-IP: 90.88.159.246 Received: from localhost (aaubervilliers-681-1-40-246.w90-88.abo.wanadoo.fr [90.88.159.246]) (Authenticated sender: maxime.ripard@bootlin.com) by relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 48DD74000A; Thu, 13 Jun 2019 13:26:04 +0000 (UTC) Date: Wed, 12 Jun 2019 17:43:25 +0200 From: Maxime Ripard To: Frank Lee Cc: rui.zhang@intel.com, Eduardo Valentin , Daniel Lezcano , robh+dt@kernel.org, Mark Rutland , Chen-Yu Tsai , David Miller , Mauro Carvalho Chehab , Greg Kroah-Hartman , Linus Walleij , Nicolas Ferre , paulmck@linux.ibm.com, Linux PM , devicetree@vger.kernel.org, Linux ARM , Linux Kernel Mailing List Subject: Re: [PATCH v3 1/3] thermal: sun8i: add thermal driver for h6 Message-ID: <20190612154325.m6z7xsxlpdq4wkxv@flea> References: <20190525181329.18657-1-tiny.windzz@gmail.com> <20190525181329.18657-2-tiny.windzz@gmail.com> <20190527122752.uc7q6zkjti3zag4q@flea> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="oy7r5ps7iphlasam" Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --oy7r5ps7iphlasam Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Jun 07, 2019 at 09:34:44PM +0800, Frank Lee wrote: > On Mon, May 27, 2019 at 8:27 PM Maxime Ripard wrote: > > > + ret = devm_request_threaded_irq(dev, irq, NULL, > > > + tmdev->chip->irq_thread, > > > + IRQF_ONESHOT, "ths", tmdev); > > > + if (ret) > > > + return ret; > > > > Is there any particular reason to use a threaded interrupt? > > Just to improve real-time. What do you mean by real-time here? If anything, that will increase the latency of the interrupts here. And in preempt-rt, regular top-half interrupts will be forced into a threaded interrupt anyway. > > > +static int sun8i_ths_remove(struct platform_device *pdev) > > > +{ > > > + struct ths_device *tmdev = platform_get_drvdata(pdev); > > > + > > > + clk_disable_unprepare(tmdev->bus_clk); > > > > I know that we discussed that already, but I'm not sure why you switch > > back to a regular call to regmap_init_mmio, while regmap_init_mmio_clk > > will take care of enabling and disabling the bus clock for you? > > It seems that regmap_init_mmio_clk just get clk and prepare clk > but no enable. At init time, yes. But it will enable it only when you access the registers, which is what you want anyway. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --oy7r5ps7iphlasam Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXQEdnQAKCRDj7w1vZxhR xXCpAQCYtE67+V1lgKQJZxd7NFi0spq61ecHzscNxJ/t6H+OXQD/ZYw/N1ST4pg3 MGrcJRd4vnNOs5alaAicdNFepVsmQQQ= =g51D -----END PGP SIGNATURE----- --oy7r5ps7iphlasam--