On 17/11/2022 17:12, Gerald Loacker wrote: >> >>> + >>> + compatible: >>> + const: ti,tmag5273 >>> + >>> + reg: >>> + maxItems: 1 >>> + >>> + "#io-channel-cells": >>> + const: 1 >>> + >>> + ti,angle-enable: >>> + description: >>> + Enables angle measurement in the selected plane. >>> + 0 = OFF >>> + 1 = X-Y (default) >>> + 2 = Y-Z >>> + 3 = X-Z >> >> This feels like something we should be configuring at runtime rather that >> DT, or is it driven by board design or similar? >> > > We use this sensor for a zoom wheel application, there is an EVM from TI > for this as well. So this is for setting the mounting position of the wheel. That's ok, but does not explain why choice of angle measurement should be a property of the hardware. I could imagine configuring device to measure sometimes X-Y and sometimes X-Z, depending on the use case. Use case can change runtime. Best regards, Krzysztof
