Hi Wilken, > Wouldn't it make more sense to add the "allwinner,sun20i-d1-hwspinlock" line > to > the driver in the sun6i_hwspinlock_ids struct, drop > "allwinner,sun6i-a31-hwspinlock" here in the D1 device tree and update the > yaml > file accordingly?
Thanks for looking at it. Bjorn hasn't replied yet, so I looked a bit more at the naming. I'd like to keep the A31 fallback: it's the usual pattern, other blocks in this dtsi do the same (timer, I2S, LED controller), and Conor already acked the binding in 2/3. If Bjorn prefers a driver entry instead, I'm fine to change it. > Oh, and I may be able to test it against the D1, I own a Sipeed Nezha. That would be great. You don't need FreeRTOS for it: I tested with a small Linux module that takes each lock and checks the status register. I can clean it up for the single-core D1 and send it. Thanks, Tien
