[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220603162351.69056bdf@jic23-huawei>
Date: Fri, 3 Jun 2022 16:23:51 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Markuss Broks <markuss.broks@...il.com>
Cc: linux-kernel@...r.kernel.org,
~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
Konrad Dybcio <konrad.dybcio@...ainline.org>,
Marijn Suijten <marijn.suijten@...ainline.org>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@...ainline.org>,
Song Qiang <songqiang1304521@...il.com>,
Lars-Peter Clausen <lars@...afoo.de>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>, linux-iio@...r.kernel.org,
devicetree@...r.kernel.org, linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH v5 0/5] Add support for ToF sensor on Yoshino platform
On Mon, 23 May 2022 20:53:39 +0300
Markuss Broks <markuss.broks@...il.com> wrote:
> This series adds support for the ToF proximity sensor installed on
> Yoshino devices. As part of this series, support handling the reset
> GPIO and VDD supply by the VL53L0X driver. Also stop hardcoding the
> interrupt type, since on Yoshino devices it seems that edge triggering
> doesn't work properly.
>
> Tested on Sony Xperia XZ1 (poplar).
>
> Cc: Konrad Dybcio <konrad.dybcio@...ainline.org>
> Cc: Marijn Suijten <marijn.suijten@...ainline.org>
> Cc: AngeloGioacchino Del Regno <angelogioacchino.delregno@...ainline.org>
Patches 1-4 applied to the togreg branch of iio.git.
Note I plan to rebase that tree after the char-misc (and hence IIO tree)
has been merged with mainline, so for now it is only pushed out as testing
so that 0-day can see if it can find anything we missed.
Thanks,
Jonathan
>
> v2:
> - Fix a nasty issue: turns out grouping the pinctrl makes it not apply,
> which was the main cause of edge interrupts not working correctly and
> having to use level interrupts, which caused a large amount of false
> detections.
> - handle the irq type more gracefully: if it's not provided, default
> to falling edge, but if it's provided, then use the provided one.
> v3:
> - add irq.h header (forgot to commit)
> - reword commit message (already initialized -> pre-initialized)
> v4:
> - reorder powering on and power off action (Jonathan)
> - sort pinctrls by GPIO number (Konrad)
> v5:
> - "This patch adds..." -> "Add ..." (Krzysztof)
>
> Markuss Broks (5):
> dt-bindings: proximity: vl53l0x: Document optional supply and GPIO
> properties
> proximity: vl53l0x: Get interrupt type from DT
> proximity: vl53l0x: Handle the VDD regulator
> proximity: vl53l0x: Handle the reset GPIO
> arm64: dts: qcom: msm8998-xperia: Introduce ToF sensor support
>
> .../bindings/iio/proximity/st,vl53l0x.yaml | 5 ++
> .../dts/qcom/msm8998-sony-xperia-yoshino.dtsi | 34 +++++++++++++
> drivers/iio/proximity/vl53l0x-i2c.c | 50 ++++++++++++++++++-
> 3 files changed, 88 insertions(+), 1 deletion(-)
>
Powered by blists - more mailing lists