Re: [avr-libc-dev] twi.h move

[Eric Weddington]

Joerg Wunsch wrote:

> I just came across this old mail from Ted:
>
> As Theodore A. Roth wrote:
>
>  
>
>
> > Regardless of the validity of making the move, the file in include/avr
> > should have had a place holder left which simply included the compat
> > file and issued a deprecation warning like we did when migrating files
> > down into the include/avr directory.
> >    
>
> I wonder whether it would be wise to move it back, and leave
> compat/twi.h as a placeholder (including <avr/twi.h>) only.  I feel I
> did rush that removal before 1.2 too much, only relying on that being
> mentioned on an old TODO list (without even agreeing to the change
> myself, I might add).
>  

Well, now this brings up our recent discussion about policy. If we 
follow it, <compat/*> is only for compatibility: compability with other 
compilers, or backwards compatibility.

The TWI is an on-board peripheral of the AVR and as such, I would say 
that it should go in <avr/twi.h>.

Would this be correct policy?

IIRC, the header file just contained some #defined values. Of course, it 
would make sense to eventually add an API of some sort to provide more 
functionality, but that can be done later down the road.

Any thoughts?

Eric