2023-10-16 20:19:12

by André Apitzsch

[permalink] [raw]
Subject: [PATCH] arm64: dts: qcom: msm8939-longcheer-l9100: Add proximity-near-level

Consider an object near to the sensor when their distance is about 4 cm
or below.

Signed-off-by: André Apitzsch <[email protected]>
---
arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts b/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
index 6802714fda3f..43cadc826121 100644
--- a/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
+++ b/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
@@ -139,6 +139,7 @@ magnetometer@d {
light-sensor@23 {
compatible = "liteon,ltr559";
reg = <0x23>;
+ proximity-near-level = <75>;

vdd-supply = <&pm8916_l17>;
vddio-supply = <&pm8916_l5>;

---
base-commit: 4d0515b235dec789578d135a5db586b25c5870cb
change-id: 20231016-bqm5_prox-4958ff602855

Best regards,
--
André Apitzsch <[email protected]>


2023-10-17 16:25:53

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: msm8939-longcheer-l9100: Add proximity-near-level



On 10/16/23 22:18, André Apitzsch wrote:
> Consider an object near to the sensor when their distance is about 4 cm
> or below.
>
> Signed-off-by: André Apitzsch <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Out of interest, what is it set to by default?

Konrad

2023-10-17 20:17:20

by André Apitzsch

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: msm8939-longcheer-l9100: Add proximity-near-level

Am Dienstag, dem 17.10.2023 um 18:25 +0200 schrieb Konrad Dybcio:
>
>
> On 10/16/23 22:18, André Apitzsch wrote:
> > Consider an object near to the sensor when their distance is about
> > 4 cm
> > or below.
> >
> > Signed-off-by: André Apitzsch <[email protected]>
> > ---
> Reviewed-by: Konrad Dybcio <[email protected]>
>
> Out of interest, what is it set to by default?
>
> Konrad

The default value is 0.

André

2023-10-18 08:32:46

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: msm8939-longcheer-l9100: Add proximity-near-level



On 10/17/23 22:03, André Apitzsch wrote:
> Am Dienstag, dem 17.10.2023 um 18:25 +0200 schrieb Konrad Dybcio:
>>
>>
>> On 10/16/23 22:18, André Apitzsch wrote:
>>> Consider an object near to the sensor when their distance is about
>>> 4 cm
>>> or below.
>>>
>>> Signed-off-by: André Apitzsch <[email protected]>
>>> ---
>> Reviewed-by: Konrad Dybcio <[email protected]>
>>
>> Out of interest, what is it set to by default?
>>
>> Konrad
>
> The default value is 0.
That much I could guess :) I was wondering if it meant more like "it can
detect movement from 1km away" or "disabled"

Konrad

2023-10-18 19:54:14

by André Apitzsch

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: msm8939-longcheer-l9100: Add proximity-near-level

Am Mittwoch, dem 18.10.2023 um 10:32 +0200 schrieb Konrad Dybcio:
>
>
> On 10/17/23 22:03, André Apitzsch wrote:
> > Am Dienstag, dem 17.10.2023 um 18:25 +0200 schrieb Konrad Dybcio:
> > >
> > >
> > > On 10/16/23 22:18, André Apitzsch wrote:
> > > > Consider an object near to the sensor when their distance is
> > > > about
> > > > 4 cm
> > > > or below.
> > > >
> > > > Signed-off-by: André Apitzsch <[email protected]>
> > > > ---
> > > Reviewed-by: Konrad Dybcio <[email protected]>
> > >
> > > Out of interest, what is it set to by default?
> > >
> > > Konrad
> >
> > The default value is 0.
> That much I could guess :) I was wondering if it meant more like "it
> can detect movement from 1km away" or "disabled"
>
> Konrad

I doubt that this sensor can detect anything from a distance of 50cm.

The values returned by the sensor are all non-negative, i.e. zero means
something like "disabled" as every raw proximity value would be
considered 'near' to the sensor according to the description in
iio/common.yaml.

André