Hello,
While looking through the Beaglebone code, I found this extraneous #elif directive. I don't think it justifies a GNATs submission (but I will make one, if that's the consensus.)
Earlier in the function, there's a group of '#ifdef's that I feel ought to be chained '#elif's to prevent the clock from being initialized multiple ways (if the wrong set of #defines is used.) Also, it seems that am335x_cpu_clk() is called twice, if VERBOSE_INIT_ARM is defined. However, I'm not qualified enough to know whether these are actual concerns or how to fix them, if they are.