search

msm8953-mainline / linux

Linux mainline kernel with WIP patches for msm8953 devices
Other
111 stars 59 forks source link

arm64: dts: qcom: msm8953-xiaomi-vince: add BMI120 alternative accele… #92

Closed M0Rf30 closed 1 year ago

M0Rf30 commented 1 year ago

…rometer

This commit enable Bosch BMI120 accelerometer found on some xiaomi-vince variants.

M0Rf30 commented 1 year ago

@fekz115 @alikates @z3ntu

Config fragment also updated here (BMI160) https://github.com/msm8953-mainline/linux/pull/91

fekz115 commented 1 year ago

Confirm that it is works

JIaxyga commented 1 year ago

I think it is necessary to leave the light-sensor, because it is indicated in the documentation. I mistakenly wrote light for tissot

JIaxyga commented 1 year ago

It would also be nice to check the documentation of the other sensors

M0Rf30 commented 1 year ago

I think it is necessary to leave the light-sensor, because it is indicated in the documentation. I mistakenly wrote light for tissot

Done

JIaxyga commented 1 year ago

I also suggest giving labels for sensors like in tissot. Then it will be clearer in the userspace which sensor gives feedback

JIaxyga commented 1 year ago

I also suggest giving labels for sensors like in tissot. Then it will be clearer in the userspace which sensor gives feedback

I'm not sure if this is how it works and need to check

M0Rf30 commented 1 year ago

I also suggest giving labels for sensors like in tissot. Then it will be clearer in the userspace which sensor gives feedback

I'm not sure if this is how it works and need to check

looking at the codebase there are not so many labels for sensors, a lot of them are for buttons and temp

alikates commented 1 year ago

Is this ready for merging then?

M0Rf30 commented 1 year ago

Yes