Received: by 2002:a5b:505:0:0:0:0:0 with SMTP id o5csp7236695ybp; Wed, 16 Oct 2019 05:57:34 -0700 (PDT) X-Google-Smtp-Source: APXvYqwDRBtXAevA82mnNm+dqwKROkaJm7SKYwcMDPZT0Mwd+n0FsDV67TPh9FgOg7Q6aa53Zuvq X-Received: by 2002:a05:6402:1b1a:: with SMTP id by26mr39201283edb.178.1571230654280; Wed, 16 Oct 2019 05:57:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571230654; cv=none; d=google.com; s=arc-20160816; b=zFHk2nn/c8z9LgbLyE0s3M52wzBuyB8Yz5mYMRmPkPm7QjPEu+m4aIggYUaHPa8Yyh vRgvUrNxkFBTXvOMaR5Tca/KQ7BCOw0e6WzI01TH7GSVLH+dT4w8vx0XJOwtZ6Q5nuRU n9vPwF/a2VFU1r39jhJAG6OclnzbyLm4d3cQ794/GVoZ+swMBy5azRm2up6Dw1vz5HbJ 6nVW1tI0JBHsDSzkkcCbmEEYleF+6A+W4ckOL8ziT04PNU61oCPlcLId5OuBRNlphUYF DlVKQ5rYDPr/z9SWEeBRc4q2c4DjyeUHV3j41s4eOCJtsBgRGUtAPag8tSsAIRTGKtsL HDrA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=GmuC3zn8bALphH5r2b0Q1YLWOrBX5XjvTZ9/8xPZZN4=; b=W79h3tjOLs2xxqWTUbUfP+9Vhld9TcBoN8EpOe18Y3tFYioLI5dYmFdZDeZ2fmfwXK cdPMX+EZa2d6RiXRXm29WN2ImKgeKV36ctzapr2h4vIsIKU3XKGHXVY+gxqqwSjM/Aov TlggAVFjM+exzAeqgdaMDQjEEwS82H5n0WjfHaIIHPPtyH9M6Rpdc+XjVg5de6PIuRH4 HOMqL8hAi/mzRP/bR/DXzbnjsraTCMN3y0Z1FIxEO6XqqzL66VLDl+oMYqMtVB5FJNhp ZKgXGpbBgUh83A8NAIpODX+WX6A73JfWahrBIOhYn8haOiPryr6CQ9Ni01NrLjT+9gFH MSFQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=ezJ2leyw; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h23si15910222ejc.1.2019.10.16.05.57.10; Wed, 16 Oct 2019 05:57:34 -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; dkim=pass header.i=@linaro.org header.s=google header.b=ezJ2leyw; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2403769AbfJPIfc (ORCPT + 99 others); Wed, 16 Oct 2019 04:35:32 -0400 Received: from mail-qk1-f194.google.com ([209.85.222.194]:46312 "EHLO mail-qk1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2391102AbfJPIfc (ORCPT ); Wed, 16 Oct 2019 04:35:32 -0400 Received: by mail-qk1-f194.google.com with SMTP id e66so1409272qkf.13 for ; Wed, 16 Oct 2019 01:35:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=GmuC3zn8bALphH5r2b0Q1YLWOrBX5XjvTZ9/8xPZZN4=; b=ezJ2leywNfinL5V540/jApDr2GEu4Od6OcnKeW75/FAlBxgP8+aMmrW3ENWC8EVwIK bbUw8QHOWvqgI2Rn//zRyqpUTL/bgIoOrv05RXTa9DixIYgal1j03iYpAo+OCXxHXxtH B8duz5djvIIx5kmVYxdveo8+w2jLJwZDXppqJReOWwEh8AeAClRF/5D52vfw2aGa3z0Y vtbc65QrHG7hThQJpg36B1viWx0nFYWkAjoJuEQQ6gaP2+c5X7XDi1pgQdA82zDpOsVv hgd4qZSz+RrWQucvq75ZkaN4bgVA8dU6IoKzIX419wej4sUDk/qe07R8NcEFU5ms2+tr /bQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=GmuC3zn8bALphH5r2b0Q1YLWOrBX5XjvTZ9/8xPZZN4=; b=svlz7CQD7NORDEu1g/bhNPTvOME7MApZHbvnMzvzOndb7tT6VLq6ozWoyLK6AxDhaA N3V3fgVuG1Xl6FwB7KgjjfLClLmVsXuNAwWrl6umWOaftgm+BfBcF0PWd86ytuysPcYT 3J3IKw3VMx54BlZ14SNaC0ahbFSH+mM1KbaG9fqrBMU+OZn51bCzOlPVfp20RHd5qVXg HMo5Ktwg/ozYeVZIS+BK8hqmi+lFPyqFj36p4gj9eAIdPhNJnLOXUb0xhvZbStLj1sIY 4CbfF+xMmnPV65Hy3IqtbwwtaFKVBsD6IH5ZI9rwGgaI4RaX3McQSdZz6aDBew/dj5oR r1+A== X-Gm-Message-State: APjAAAXb4tSU+R9EXF758kl/P/rDCp7ffNwEnlP1gu79Is9HKkpRZzqi lzm+YdEvKFoEpIKI84uc0C6kzr9OZq2QLEYl9ftHQw== X-Received: by 2002:a37:8806:: with SMTP id k6mr39103180qkd.127.1571214930602; Wed, 16 Oct 2019 01:35:30 -0700 (PDT) MIME-Version: 1.0 References: <6e6c6da8-0948-581c-a4a3-7a2617ac5dd0@linaro.org> In-Reply-To: <6e6c6da8-0948-581c-a4a3-7a2617ac5dd0@linaro.org> From: Amit Kucheria Date: Wed, 16 Oct 2019 14:05:19 +0530 Message-ID: Subject: Re: [PATCH 00/15] thermal: qcom: tsens: Add interrupt support To: Daniel Lezcano Cc: Linux Kernel Mailing List , linux-arm-msm , Bjorn Andersson , Eduardo Valentin , Andy Gross , Brian Masney , Stephen Boyd , Amit Kucheria , Mark Rutland , Rob Herring , Zhang Rui , DTML , Linux PM list Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 16, 2019 at 1:29 PM Daniel Lezcano wrote: > > On 16/10/2019 09:33, Amit Kucheria wrote: > > Hi Thermal and MSM maintainers, > > > > I believe this series is now ready to be merged. The DT bindings and dr= iver > > changes should go through the thermal tree and the changes to the DT fi= les > > themselves should go through the MSM tree. There is no hard ordering > > dependency because we're adding a new property to the driver. It would = help > > to soak in linux-next for a few weeks to catch anything on kernelci.org= . > > So the ones going to thermal are: > > 1-7, 14, 15 right ? 1-4, 7, 14, 15 =3D> thermal tree 5, 6, 8-13 =3D> msm tree I guess I could have ordered it better for merging :-/ > > Changes since v4: > > - Change to of-thermal core[1] to force interrupts w/o changing polling= -delay DT > > parameter > > - Corresponding changes to DT files to remove the hunks setting the val= ues > > to 0 > > - Collected reviews and acks > > > > Changes since v3: > > - Fix up the YAML definitions based on Rob's review > > > > Changes since v2: > > - Addressed Stephen's review comment > > - Moved the dt-bindings to yaml (This throws up some new warnings in va= rious QCOM > > devicetrees. I'll send out a separate series to fix them up) > > - Collected reviews and acks > > - Added the dt-bindings to MAINTAINERS > > > > Changes since v1: > > - Collected reviews and acks > > - Addressed Stephen's review comments (hopefully I got them all). > > - Completely removed critical interrupt infrastructure from this series= . > > Will post that separately. > > - Fixed a bug in sign-extension of temperature. > > - Fixed DT bindings to use the name of the interrupt e.g. "uplow" and u= se > > platform_get_irq_byname(). > > > > Add interrupt support to TSENS. The first 6 patches are general fixes a= nd > > cleanups to the driver before interrupt support is introduced. > > > > [1] https://lore.kernel.org/linux-arm-msm/1b53ef537203e629328285b4597a0= 9e4a586d688.1571181041.git.amit.kucheria@linaro.org/ > > > > Amit Kucheria (15): > > drivers: thermal: tsens: Get rid of id field in tsens_sensor > > drivers: thermal: tsens: Simplify code flow in tsens_probe > > drivers: thermal: tsens: Add __func__ identifier to debug statements > > drivers: thermal: tsens: Add debugfs support > > arm: dts: msm8974: thermal: Add thermal zones for each sensor > > arm64: dts: msm8916: thermal: Fixup HW ids for cpu sensors > > dt-bindings: thermal: tsens: Convert over to a yaml schema > > arm64: dts: sdm845: thermal: Add interrupt support > > arm64: dts: msm8996: thermal: Add interrupt support > > arm64: dts: msm8998: thermal: Add interrupt support > > arm64: dts: qcs404: thermal: Add interrupt support > > arm: dts: msm8974: thermal: Add interrupt support > > arm64: dts: msm8916: thermal: Add interrupt support > > drivers: thermal: tsens: Create function to return sign-extended > > temperature > > drivers: thermal: tsens: Add interrupt support > > > > .../bindings/thermal/qcom-tsens.txt | 55 -- > > .../bindings/thermal/qcom-tsens.yaml | 168 ++++++ > > MAINTAINERS | 1 + > > arch/arm/boot/dts/qcom-msm8974.dtsi | 92 +++ > > arch/arm64/boot/dts/qcom/msm8916.dtsi | 6 +- > > arch/arm64/boot/dts/qcom/msm8996.dtsi | 4 + > > arch/arm64/boot/dts/qcom/msm8998.dtsi | 6 +- > > arch/arm64/boot/dts/qcom/qcs404.dtsi | 2 + > > arch/arm64/boot/dts/qcom/sdm845.dtsi | 4 + > > drivers/thermal/qcom/tsens-8960.c | 4 +- > > drivers/thermal/qcom/tsens-common.c | 529 ++++++++++++++++-- > > drivers/thermal/qcom/tsens-v0_1.c | 11 + > > drivers/thermal/qcom/tsens-v1.c | 29 + > > drivers/thermal/qcom/tsens-v2.c | 13 + > > drivers/thermal/qcom/tsens.c | 58 +- > > drivers/thermal/qcom/tsens.h | 286 ++++++++-- > > 16 files changed, 1102 insertions(+), 166 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/thermal/qcom-tsen= s.txt > > create mode 100644 Documentation/devicetree/bindings/thermal/qcom-tsen= s.yaml > > > > > -- > Linaro.org =E2=94=82 Open source software for A= RM SoCs > > Follow Linaro: Facebook | > Twitter | > Blog >