<div dir="ltr">To Joel in particular,<div><br></div><div>Thanks for making a WDT driver for the Aspeed chips already! It makes diving into these things a lot easier. </div><div><br></div><div>I didn't understand some of the things in the aspeed-g5.dtsi file though:</div><div><br></div><div><div>wdt1: wdt@1e785000 {</div><div><span class="gmail-Apple-tab-span" style="white-space:pre"> </span>compatible = "aspeed,ast2500-wdt";</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">   </span>reg = <0x1e785000 0x1c>;</div><div>};</div><div><br></div><div>wdt2: wdt@1e785020 {</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">    </span>compatible = "aspeed,ast2500-wdt";</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">   </span>reg = <0x1e785020 0x1c>;</div><div><span class="gmail-Apple-tab-span" style="white-space:pre"> </span>status = "disabled";</div><div>};</div><div><br></div><div>wdt3: wdt@1e785040 {</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">    </span>compatible = "aspeed,wdt";</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">   </span>reg = <0x1e785074 0x1c>;</div><div><span class="gmail-Apple-tab-span" style="white-space:pre"> </span>status = "disabled";</div><div>};</div></div><div><br></div><div><br></div><div><div>  * Register lengths are 0x1C instead of 0x20. The skipped register in the datasheet for AST2500 is determining which hw modules to reset, which sounds like something we'd want to configure.</div><div>  * wdt3 compatible string is different. </div><div>  * wdt3 register offset appears wrong.</div></div><div><br></div><div>Looking to get some feedback. Thanks!</div><div><br></div><div>Nolan</div><div><br></div></div>