[avr-libc-dev] RE: [bug #17470] Add API for CLKPR register to <avr/power.h>

[Eric Weddington]

> -----Original Message-----
> From: Joerg Wunsch [mailto:address@hidden 
> Sent: Thursday, August 24, 2006 1:07 PM
> To: Joerg Wunsch; Eric Weddington; address@hidden
> Subject: [bug #17470] Add API for CLKPR register to <avr/power.h>
> 
> 
> Follow-up Comment #4, bug #17470 (project avr-libc):
> 
> > What about making this <avr/clock.h> instead?
> 
> Rather not.  I'd even have preferred it to combine the power save
> register and sleep mode stuff into a single file rather than
> two; 

Except that it was already called sleep.h; it didn't make sense to put
anything else in sleep.h that really doesn't have to do with sleeping.

> it doesn't make much sense to start one header file per
> control register.  

I agree; I'm just trying to group things logically with their names. If
changing the system clock rate is only used for power savings, then I agree
100% it should be put in power.h. My only concern is if it is used *in some
other way* other than for power savings.

As things stand, I'll put it in power.h.

> Other compilers/libraries ship with about
> two or three header files for everything, we already have more
> than a dozen of them in the avr/ subdir.

It's a feature, not a bug. We have fine-grained control. ;-)

Thanks
Eric