[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[avr-libc-dev] RE: [bug #17470] Add API for CLKPR register to <avr/power
From: |
Eric Weddington |
Subject: |
[avr-libc-dev] RE: [bug #17470] Add API for CLKPR register to <avr/power.h> |
Date: |
Thu, 24 Aug 2006 13:23:59 -0600 |
> -----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