On Thu, Feb 27, 2025 at 07:35:09AM -0600, Rob Herring wrote: > On Thu, Feb 27, 2025 at 07:42:29AM +0800, Inochi Amaoto wrote: > > Some SoCs from Aspeed, Allwinner, Sophgo and Synopsys have > > a simple reset controller by toggling bit. It is a hard time > > for each device to add its own compatible to the driver. > > It's a 1 line change. That isn't hard. > Yeah, it is not hard. But it is annoying and sometimes I felt it is a waste of time to add a device id to the reset-simple driver. > > Since these devices share a common design, it is possible to > > add a common device to reduce these unnecessary change for > > the driver. > > > > Add common binding for these kind generic reset controller. > > Check the binding description for its requirement and > > suitable scenarios. > > We generally don't want 1 register per node type bindings. That level of > detail in DT has proven impossible to get right. > > Rob It is not 1 register per node, but a contiguous range. In most SoC satisfy the requirement, only one reset device is added. This is what the reset-simple handle. 1 register per node is too crazy for everyone.... Regards, Inochi