This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: at91 cdl refactoring , settable baudrate from redboot


Thomas Koeller wrote:
Jonathan Larmour wrote:

I'll hold off reverting the relevant parts of the patch (i.e. return bits back to the platform HALs) straight away so people have a chance to comment.



Jonathan,


do you think you need much longer to make a decision as to whether this patch should be reverted or not?

I think it should be reverted, but I don't have time right this second to do it, sorry. (Hard disk failure, and some deadlines are a lot tighter).


Of course if you wanted to submit the patch yourself, that would be fine too; but otherwise it's likely to be next week for me.

As I wrote earlier, I have a couple of platform HAL packages that would need to be adopted if this change were to stay, but of course I do not want to do that if things are likely to be changed back later. But if I dont, I cannot update my repository.

FWIW, I agree with you that most of the cdl options in question should be in the platfrom HAl for maximum flexibility. The platform HAL packages I am maintaining are not exactly like one of the evaluation boards. Some of them have additional I/O channels besides the two serial ports, and they do in fact use a special startup mode, which is a variant of the ROMRAM mode that leaves .rodata in rom.

Yep, that justifies the reasoning :).


Jifl
--
eCosCentric    http://www.eCosCentric.com/    The eCos and RedBoot experts
>>>>> Visit us in booth 2527 at the Embedded Systems Conference 2004 <<<<<
March 30 - April 1, San Francisco http://www.esconline.com/electronicaUSA/
--["No sense being pessimistic, it wouldn't work anyway"]-- Opinions==mine


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]