From: SourceForge.net <no...@so...> - 2012-06-27 16:54:09
|
Patches item #3537365, was opened at 2012-06-23 07:25 Message generated for change (Comment added) made by molnarkaroly You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300599&aid=3537365&group_id=599 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Molnár Károly (molnarkaroly) Assigned to: Nobody/Anonymous (nobody) Summary: Pic14 and pic14e libio (adc,i2c,pwm,spi,usart) Initial Comment: As I promised, this the pic14 and pic14e libio. It includes adc, i2c, pwm, spi and usart support. Of course, only they the devices which are knows the proper hardware function. (I tried to do a flawless job, we'll see how well.) ---------------------------------------------------------------------- >Comment By: Molnár Károly (molnarkaroly) Date: 2012-06-27 09:54 Message: I think it would be nice, but I became uncertain due to comment of tecodev. Maybe it would be better separate the directories from the compiler. Somehow should be published and then people decide on they need it or not. ---------------------------------------------------------------------- Comment By: Diego Herranz (diegoherranz) Date: 2012-06-27 06:56 Message: Talking about libraries to be or not to be included in the compiler, I have an eeprom library for PIC16 which I was about to submit and then I found this topic. Would you consider it useful/appropriate for SDCC to have it included? ---------------------------------------------------------------------- Comment By: Molnár Károly (molnarkaroly) Date: 2012-06-23 11:01 Message: Actually, I wanted to create a support is like pic16 branch. I did not want more. > I am also not sure whether the existing libraries are used by a sufficiently large number of people to justify the effort. I see. Maybe it's not worth it at all. ---------------------------------------------------------------------- Comment By: Raphael Neider (tecodev) Date: 2012-06-23 10:36 Message: > What is the obstacle? The obstacle is mostly the purpose of the sdcc project. We want to provide a C compiler for small devices. We do not want to provide a complete libc (but we more or less do ...). And we do not primarily want to provide a large amount of library code, which we would have to maintain as well. The question of whether the library is better integrated into the compiler (as is done, e.g., with the Microchip compilers, if I am not mistaken) or whether it should be put into a separate project is difficult to answer. For the PIC16 target, someone (might have been me, not sure) opted to add an I/O library, which caused a fair amout of headache due to the numerous mutually incompatible devices. The headaches continue with each device for which support is added to the compiler, as the complete I/O lib may have to be extended as well. I am also not sure whether the existing libraries are used by a sufficiently large number of people to justify the effort. After all, the API (especially ADC) has become pretty complex with all these #if "your device fits here" blocks that tell you, what symbols you may use. I have not yet looked at this patch; generally I would vote in favour of accepting I/O libraries. To cut a long story short: Adding or not adding the library to the SDCC sources is more a political decision than it is a technical one. > Too large of the patch, or I sent too much lately? Large patches and numerous patches might cause delays in the acceptance process, but do not cause rejection of the contributions. ---------------------------------------------------------------------- Comment By: Molnár Károly (molnarkaroly) Date: 2012-06-23 09:56 Message: > We will have to decide whether the io library should be a part of SDCC or > should be published elsewhere as stand alone library. What is the obstacle? Too large of the patch, or I sent too much lately? ---------------------------------------------------------------------- Comment By: Borut Ražem (borutr) Date: 2012-06-23 07:47 Message: Thanks Karoly! This is a candidate to be included after the SDCC 3.2.0 release. We will have to decide whether the io library should be a part of SDCC or should be published elsewhere as stand alone library. Currenly SDCC includes io library only for pic16 targets. The same concern applies for delay functions you submitted in patch #3526654. Borut ---------------------------------------------------------------------- Comment By: Molnár Károly (molnarkaroly) Date: 2012-06-23 07:34 Message: Too late I noticed that the patch is included in the sdcc-3.2.0.spec well. I do not wanted to send this. With this I create rpm package myself. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300599&aid=3537365&group_id=599 |