19:26 <macromorgan> mmind00: Can you confirm this should be SWITCH_REG1 and not just SWITCH_REG? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/rk3188-px3-evb.dts?h=v5.17-rc6#n215

19:27 <macromorgan> the yaml schema I'm pushing flagged that as a discrepancy, so I checked the driver source code for the rk808-regulator.c and it looks like it's wanting a SWITCH_REG rather than a SWITCH_REG1. If it should be changed I'll add this change to my patch series.

20:24 <mmind00> macromorgan: no clue ...

20:24 <mmind00> - I see that the rk3288-phycore also uses the _rk818_ (as the board in question), and already uses the SWITCH_REG without number

20:24 <mmind00> - the px3-evb was submitted by a Rockchip engineer, so having the SWITCH_REG with "1" in there probably is a remnant from the vendor-kernek

20:24 <mmind00> - I see that boards using the rk808 have multiple SWITCH_REG{1,2} regulators, so on them that would be obviously justified

20:25 <mmind00> so bottom line: I guess this should be changed then

20:25 <mmind00> (in the px3-evb.dts)

20:25 <macromorgan> okay... I'm going to update it to SWITCH_REG as part of the series for the yaml-izing of the documentation

20:25 <macromorgan> apparently I didn't have the latest and greatest dt-schema, so I missed 2 errors in the last submission

20:26 <macromorgan> now I have the most recent version and yamllint, dt_binding_check and dtbs_check all come out clean for both arm and arm64

20:26 <mmind00> yay

20:27 <macromorgan> yep... I was asked to do this before I submitted the battery driver. Of course I keep identifying bugs as the battery driver gets tested, so who knows when that will be...