Describe upstream regulators for Witherspoon's BMP280
amboar opened this issue · comments
Currently the following warnings are issued for Witherspoon's devicetree:
arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dt.yaml: bmp280@77: 'vddd-supply' is a required property
arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dt.yaml: bmp280@77: 'vdda-supply' is a required property
FYI, that device only existed on some early lab versions of that system, and didn't ship to the field.
Can we drop it from the devicetree then?
@amboar, I'd prefer that it does not get dropped as that would then cause these lab systems still in use to no longer have an available ambient sensor.
Not that I am a kernel guru or anything, but BMP280 issue caught my eye.
I was curious what those were, and why they would be required.
Is it just me, or did those start out as optional and move to required?
https://patchwork.ozlabs.org/patch/637124/
@bjwyman hah! Indeed they did, but not in the change you pointed to, but rather this one:
This was introduced in v5.2. I'll poke people.
Okay, I was just noting that going back to the patch I mentioned, those appeared to be optional, but somehow they later ended up as required. The change you pointed to is obviously where that was introduced.