Re: Serial driver for ARM s3c4510

[Jonathan Larmour <jifl at eCosCentric dot com>]

Bart Veer wrote:
> The CDL restriction enforces package encapsulation. Each package can
> be sure of the contents of its own <pkgconf/whatever> header, nothing
> else can interfere with that contents and there are never any concerns
> about the order in which things might appear in that header. I would
> need a much more persuasive argument before changing this aspect of
> CDL.

By that token, having packages dump stuff in <pkgconf/system.h> breaks 
that encapsulation just as much, if not more because of the widespread 
nature. Anything can interfere with that (and define or undefine any 
options), and the order could clearly be important. And as I said, doing 
so causes effectively a whole system rebuild unnecessarily.

Perhaps the use of "puts $::cdl_system_header" should be deprecated. Being 
able to define something in a parent package's header is at best no worse 
than that functionality, and more likely less bad.

The alternative is to make new booldata CDL options, which requires wider 
changes, and no-one is likely to change current practice if 
$::cdl_system_header is available and not deprecated.

Jifl
--
eCosCentric    http://www.eCosCentric.com/    The eCos and RedBoot experts
--["No sense being pessimistic, it wouldn't work anyway"]-- Opinions==mine