From: SourceForge.net <no...@so...> - 2009-05-30 15:21:37
|
Feature Requests item #1930974, was opened at 2008-04-01 09:44 Message generated for change (Settings changed) made by borutr You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=350599&aid=1930974&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: Closed Priority: 5 Private: No Submitted By: Patryk (patryks) Assigned to: Borut Ražem (borutr) Summary: Some device specific header files are not in their folders Initial Comment: There are some device specific header files in SDCC\include that could be moved to suitable folders: - 80c51xa.h, ds80c390.h, ds400rom.h, ser.h, ser_ir.h, serial390.h, serial.h, tinibios.h ---> mcs51, - z180.h ---> z80. ---------------------------------------------------------------------- >Comment By: Borut Ražem (borutr) Date: 2009-05-30 17:21 Message: Fixed in svn revision #5465. I moved device/include/mcs51/ds80c390.h and device/include/mcs51/tinibios.h back to device/include, since they are used also by ds400, as proposed by Maarten. But I created ds400 directory and put ds400rom.h, since the ds400 target is serching for includes in include/ds400. This time I even ran the regression tests on my machine ;-) Borut ---------------------------------------------------------------------- Comment By: Maarten Brock (maartenbrock) Date: 2009-05-30 15:24 Message: If you create a ds390 directory, I would leave it at that and not also add a ds400 directory or even tininative directory. They are all handled by the ds390 target. So all 3 headers can go into the same ds390 dir. But the include search logic must be changed to accomodate this, because currently it searches in a subdir with the target name as given after the -m switch. The easy alternative is to revert and put all these files back in the root of include. ---------------------------------------------------------------------- Comment By: Borut Ražem (borutr) Date: 2009-05-30 12:08 Message: I have an additional concern / question: do tinibios.h and ds80c390.h belong only to ds390 or they are used also by ds400? I saw comments in tinibios.h which make ma believe that it is used also by ds400. And tinybios.h includes ds80c390.h... If this is true, tinibios.h and ds80c390.h should remain in the SDCC/include directory or we should make a copy also in ds400 directory. I there anybody who can answer ma question? What do you think it would be the best solution? Borut ---------------------------------------------------------------------- Comment By: Maarten Brock (maartenbrock) Date: 2009-05-28 12:33 Message: That would be an apropriate solution. But I decided against it when I created the mcs51 subdirectory because it concerns such a small number of files. But more problematic: The ds390 regression test now fails because it cannot find its includes! ---------------------------------------------------------------------- Comment By: Borut Ražem (borutr) Date: 2009-05-25 21:31 Message: Should we create include/ds390 and include/xa51 directories? Borut ---------------------------------------------------------------------- Comment By: Maarten Brock (maartenbrock) Date: 2009-05-25 12:17 Message: I'm sorry I did not reply to this item before, but I disagree with this request and vote to revert the change. The mentioned files are for the DS390 or XA target which have their own target specific backend. Therefor they do not belong in the mcs51 directory IMHO. Maarten ---------------------------------------------------------------------- Comment By: Borut Ražem (borutr) Date: 2009-05-25 07:23 Message: Implemented in svn revision #5463. Borut ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=350599&aid=1930974&group_id=599 |