The chws array in drivers/clk/at91/sam9x60.c however gets main_rc_osc (from clock generator), mainck (clock generator), pllack (clock generator), upllck (clock generator, UTMI), but also mck (from PMC). This creates the impression things are mixed up here. I find all this quite confusing to be honest. > The other "#define PMC_*" defines will eventually go to SoC specific > bindings. "#define AT91_PMC_*" seems to not belong here anyway and these > would in the end removed, as well. Okay, you seem to have an idea how this should look like in the long run. Are there any plans at Microchip or at91 clock maintainer side to clean this up in the near future? I would like to rather put my small changes for otpc on top of a clean tree, instead of trying to clean up clock drivers and bindings for a whole family of SoCs and boards, where I can test only one of them. O:-) Greets Alex