On Thu, Oct 24, 2024 at 06:09:01PM +0800, Tony Chung wrote: > This patch adds define for CLOCK_SELECT_REG1 & CLOCK_SELECT_REG2 offsets. > These two registers can select clock source between 30M/96M/External. I think should just add these as part of the patch that uses them. If you have some information on how they are used, that would be useful to have in the commit message and possibly also in a comment above the defines. Where did you learn about these settings? Johan