Re: [PATCH 3/4] ARM: dts: qcom: msm8960: expressatt: Add Magnetometer

[Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>]

On 11/21/25 12:35 PM, Rudraksha Gupta wrote:
>>> +        iovdd-supply = <&pm8921_lvs4>;
>>> +
>>> +        // TODO: Figure out Mount Matrix
>> pmOS wiki has this:
>>
>> https://github.com/SiddharthManthan/accelerometer-mount-matrix-generator/blob/master/rotation_matrix.py
>>
>> but for accelerometers.. shouldn't be too different, I would assume
> 
> Unfortunately this seems to generate a broken matrix for me for the accelerometer. Downstream is also useless here as I believe Android does something slightly different.
> 
> I have been trying to upstream the pinephone pro's accelerometer and magnetometer as well ( https://lore.kernel.org/all/20250929-ppp_light_accel_mag_vol-down-v4-0-6598f22d3451@gmail.com/ ), but I'm just not sure if what I'm doing is right. I think the easiest solution for me right now is to get the screen working and use GUI programs to figure out if the mount matrix is right. If you have other suggestions, please let me know. I'm all ears.
> 
> But since I have your attention for now, I had a couple questions on trying to get the screen to work:
> 
> - The pipeline is MDP4 <-> DSI <-> Panel, right? Any information you can give me so that I can try to get this working faster?

Yes that's the pipeline

Unless the MDP4 driver is missing something (which it may as well, all
hw since ~2013 has moved on to MDP5/DPU), you should just be required to
come up with a panel driver (DSI init sequences, poking at regulators,
reset GPIO(s) etc.)

> - Is IOMMU broken? It seems like I can use APQ8064's implementation, but that's broken since at least 5.10. Should this be reported, and if so where?

I don't know, this platform uses its own arcane driver.. +Dmitry?
If you have a bug report, please send it to linux-arm-msm

> - I see Adreno 2xx, but seems like there is not Adreno 225 DTS compatible? Any insight here?

Adreno is a separate HW block, you can/should get software rendering to
work first

Konrad