<div dir="ltr">OMG, I'm sorry. Ignore. I got confused between ASPEED_CLK_* being random identifiers and ASPEED_RESET_* actually being bits in the registers. Sorry.<div><br></div><div>Oskar.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 12, 2020 at 7:22 PM Oskar Senft <<a href="mailto:osk@google.com">osk@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hey Joel and Andrew<div><br></div><div>I'm working on the AST2600 EVB trying to get some eSPI functionality to work using <a href="https://github.com/AspeedTech-BMC/linux/blob/aspeed-dev-v5.1/drivers/soc/aspeed/aspeed-espi.c" target="_blank">https://github.com/AspeedTech-BMC/linux/blob/aspeed-dev-v5.1/drivers/soc/aspeed/aspeed-espi.c</a>. The driver is missing support for clock enablement so I dug into how this could be added.</div><div><br></div><div>I noticed that you guys worked on include/dt-bindings/clock/ast2600-clock.h but from what I can tell the ASPEED_CLK_GATE_* definitions have not yet been updated from AST2500 to AST2600.</div><div><br></div><div>Am I looking at the latest version of the file? I'm happy to update the clock gate bits, but I don't want to duplicate work that you have in the pipeline.</div><div><br></div><div>Thanks</div><div>Oskar.</div></div>
</blockquote></div>