gnupic: Thread: Re: [gnupic] error building gpsim extra graphic_lcd module

Xiaofan Chen wrote:
> I just found out there are some extra modules not built at the
> same time with gpsim. I have problem with the graphic_lcd module.

<snip>

> /usr/local/include/gpsim/ValueCollections.h:299: error: there are no
> arguments to 'rpl_malloc' that depend on a t emplate parameter, so a
> declaration of 'rpl_malloc' must be available

Hi Xiaofan,

I suspect that when you build gpsim that you don't see this error, 
correct? The line in question is type casting a call to malloc(). 
However, I don't see that stdio.h has been included. So, could you try 
adding:

   #include <stdio.h>

At the top of ValueCollections.h and see if this makes the error go 
away? Or maybe a more expedient check would be to add a #include 
<stdio.h> is sed.cc before the point where gpsim/ValueCollections.h is 
included.

Also, I think this inlined code needs to be moved to a file. I'm pretty 
sure that the graphics LCD module is not making use of it.

Scott

On 5/23/06, Scott Dattalo ####@####.#### wrote:
> Xiaofan Chen wrote:
> > I just found out there are some extra modules not built at the
> > same time with gpsim. I have problem with the graphic_lcd module.
>
> <snip>
>
> > /usr/local/include/gpsim/ValueCollections.h:299: error: there are no
> > arguments to 'rpl_malloc' that depend on a t emplate parameter, so a
> > declaration of 'rpl_malloc' must be available
>
> Hi Xiaofan,
>
> I suspect that when you build gpsim that you don't see this error,
> correct?

Yes there are no errors when building gpsim.

>The line in question is type casting a call to malloc().
> However, I don't see that stdio.h has been included. So, could you try
> adding:
>
>    #include <stdio.h>
>
> At the top of ValueCollections.h and see if this makes the error go
> away? Or maybe a more expedient check would be to add a #include
> <stdio.h> is sed.cc before the point where gpsim/ValueCollections.h is
> included.
>

Thanks for pointing out the cause.

<stdio.h> is included in the file sed1520.cc but not ValueCollections.h.

It turns out that we need to include <malloc.h> under Linux. So if
I add
        #include <malloc.h>
to the file ValueCollextions.h and the building will be okay, I hope this
is the correct fix. I also think this may break Windows so conditional define
may be necessary.


Regards,
Xiaofan

After building the graphic_lcd module, I find another problem with
the example. The stc files (startup.stc and glcd_test.stc) are missing in the
graphic_lcd/examples folder.

By the way, the extra/lcd module works fine.

Regards,
Xiaofan

Xiaofan Chen wrote:
> After building the graphic_lcd module, I find another problem with
> the example. The stc files (startup.stc and glcd_test.stc) are missing 
> in the
> graphic_lcd/examples folder.

There are no .stc files in the graphic_lcd/examples. All of the 
simulator configuration is built into the source files with gpasm .sim 
directives. Look at glcd_test.asm. Also, there is an update bug with the 
graphic LCD module. To refresh the LCD screen, you have to halt the 
simulation and click on the module.

Scott

On 5/25/06, Scott Dattalo ####@####.#### wrote:
>
> There are no .stc files in the graphic_lcd/examples. All of the
> simulator configuration is built into the source files with gpasm .sim
> directives. Look at glcd_test.asm. Also, there is an update bug with the
> graphic LCD module. To refresh the LCD screen, you have to halt the
> simulation and click on the module.
>

Sorry I was looking at the Makefile and I though "make sim" would do
the job for me. It didn't.

I tried "gpsim -c glcd_test.cod" and there is no graphic LCD windows.
If I try the extras/lcd module, an LCD screen will pop up.
How should I test if my build of graphic_lcd is working?


mcuee@ubuntu:~/Desktop/build/gpsim/gpsim/extras/graphic_lcd/examples$ gpsim
gpsim - the GNUPIC simulator
version: Release 0.21.12-pre


type help for help
**gpsim>
(gpsim:27022): Gdk-WARNING **: locale not supported by Xlib

(gpsim:27022): Gdk-WARNING **: cannot set locale modifiers
SimulationMode:52

**gpsim> Warning SSP Module is not implemented
FIXME: 16bit processor is assuming that PLL is on - should check config bits
FIXME:  HLL files are not supported at the moment
SSPCON1 is not implemented
SSPCON2 is not implemented
SSPSTAT is not implemented
SSPADD is not implemented
Program_Counter16::put_value 0x0
Enabling WDT  timeout = 0.018 seconds
pullup resistor or generic voltage source
 Attributes:
  .resistance - pullup resistance
  .voltage - pullup or drive voltage
 .capacitance - pin capacitance

pullup resistor or generic voltage source
 Attributes:
  .resistance - pullup resistance
  .voltage - pullup or drive voltage
 .capacitance - pin capacitance

0: p18f452 Execution at done(0x298)
    Message:*** PASSED LCD test
1: p18f452 Execution at 0x29c
    Message:*** FAILED LCD test
NSourcePage::setSource() - view=0x846c0c8
NSourcePage::setSource() - view=0x84834b8
Received button press for view 0x81e3a00
 type=4 x=216,y=370
**gpsim> FIXME: CLRWDT for 16 bit processors
NSourcePage::setSource() - view=0x84b66e0
0x00000000014FB17E p18f452 0x01E2 0x948D bcf    late,2,0
0x00000000014FB17F p18f452 0x01E4 0xD80B rcall  $+0x18  ;(0x001fc)
Widths 36 0 0 0
module_x 0, module widget size 316, 127
Widths 36 0 0 0
Widths 36 0 0 0
Widths 36 0 36 0
...


Regards,
Xiaofan

On 5/25/06, Xiaofan Chen ####@####.#### wrote:
> I tried "gpsim -c glcd_test.cod" and there is no graphic LCD windows.
> If I try the extras/lcd module, an LCD screen will pop up.
> How should I test if my build of graphic_lcd is working?

Apparently I have some problems with the gpsim installation (the
breadboard module is not working). The graphic_lcd module
(http://www.dattalo.com/gnupic/glcd.png) looks very nice. The
breadboard module under my Linux installation is not correct
at all.

I then tried gpsim-win32 snapshot and I can open the breadboard.
Unfortunately it is very hard to build the extra module under Windows
(I could not build gpsim under Windows anyway). Therefore the
graphic_lcd is missing.

It would be nice that the extra modules are built in the Win32
snapshot.

Regards,
Xiaofan

On Thu, 2006-05-25 at 20:00 +0800, Xiaofan Chen wrote:

>I tried "gpsim -c glcd_test.cod" and there is no graphic LCD windows.

What happens when you try

  gpsim -s glcd_test.cod

Scott

On 5/25/06, Scott Dattalo ####@####.#### wrote:
> On Thu, 2006-05-25 at 20:00 +0800, Xiaofan Chen wrote:
>
> >I tried "gpsim -c glcd_test.cod" and there is no graphic LCD windows.
>
> What happens when you try
>
>   gpsim -s glcd_test.cod
>

The first time I saw a splash screen with graphic LCD and then gpsim
caused a segmentation fault.

mcuee@ubuntu:~/Desktop/build/gpsim/gpsim/extras/graphic_lcd/examples$
gpsim -s g lcd_test.cod

gpsim - the GNUPIC simulator
version: Release 0.21.12-pre


type help for help
**gpsim>
(gpsim:8633): Gdk-WARNING **: locale not supported by Xlib

(gpsim:8633): Gdk-WARNING **: cannot set locale modifiers
SimulationMode:52
Warning SSP Module is not implemented
FIXME: 16bit processor is assuming that PLL is on - should check config bits
Widths 36 0 36 0
FIXME:  HLL files are not supported at the moment
SSPCON1 is not implemented
SSPCON2 is not implemented
SSPSTAT is not implemented
SSPADD is not implemented
Program_Counter16::put_value 0x0
Enabling WDT  timeout = 0.018 seconds
Widths 36 0 0 0
module_x 0, module widget size 316, 127
pullup resistor or generic voltage source
 Attributes:
 .resistance - pullup resistance
 .voltage - pullup or drive voltage
 .capacitance - pin capacitance

Widths 36 0 0 0
pullup resistor or generic voltage source
 Attributes:
 .resistance - pullup resistance
 .voltage - pullup or drive voltage
 .capacitance - pin capacitance

Widths 36 0 0 0
0: p18f452 Execution at done(0x298)
    Message:*** PASSED LCD test
1: p18f452 Execution at 0x29c
    Message:*** FAILED LCD test
**gpsim> Segmentation fault


After that I can did not see the LCD and gpsim segfaults if I want to
load the breadboard.

I will do a fresh checkout of the svn source and then build gpsim
and the module from scratch later.

Regards,
Xiaofan

> The first time I saw a splash screen with graphic LCD and then gpsim
> caused a segmentation fault.

I could duplicate this on one system. However, I'm away that computer now
and unable to debug the problem (and I won't be able to get back to it for
a few days). However, I did find some data members that were not getting
initialized and this may be the cause. Also, there may've been a race
condition between when GTK initialized a graphical element and the point
where gpsim assumed it was initialized. I added a check for that. The code
is in SVN.

Scott

On 5/26/06, Scott Dattalo ####@####.#### wrote:
>
> I could duplicate this on one system. However, I'm away that computer now
> and unable to debug the problem (and I won't be able to get back to it for
> a few days). However, I did find some data members that were not getting
> initialized and this may be the cause. Also, there may've been a race
> condition between when GTK initialized a graphical element and the point
> where gpsim assumed it was initialized. I added a check for that. The code
> is in SVN.
>

With the latest SVN version of gpsim, the graphic_lcd module can be built
under Linux without any changes. However the segfault problem remains.

mcuee@ubuntu:~/Desktop/build/gpsim/gpsim/extras/graphic_lcd/examples$
gpsim -s glcd_test.cod

gpsim - the GNUPIC simulator
version: Release 0.21.12-pre


type help for help
**gpsim>
(gpsim:4270): Gdk-WARNING **: locale not supported by Xlib

(gpsim:4270): Gdk-WARNING **: cannot set locale modifiers
SimulationMode:52
Warning SSP Module is not implemented
FIXME: 16bit processor is assuming that PLL is on - should check config bits
Setting config word 0x300000 = 0xdeff
Setting config word 0x300002 = 0xffff
Setting config word 0x300004 = 0xfffb
Setting config word 0x300006 = 0xfffb
Setting config word 0x300008 = 0xbffc
Setting config word 0x30000a = 0xffff
Setting config word 0x30000c = 0xffff
FIXME:  HLL files are not supported at the moment
SSPCON1 is not implemented
SSPCON2 is not implemented
SSPSTAT is not implemented
SSPADD is not implemented
Program_Counter16::put_value 0x0
Enabling WDT  timeout = 0.018 seconds
pullup resistor or generic voltage source
 Attributes:
  .resistance - pullup resistance
  .voltage - pullup or drive voltage
 .capacitance - pin capacitance

pullup resistor or generic voltage source
 Attributes:
  .resistance - pullup resistance
  .voltage - pullup or drive voltage
 .capacitance - pin capacitance

0: p18f452 Execution at done(0x298)
    Message:*** PASSED LCD test
1: p18f452 Execution at 0x29c
    Message:*** FAILED LCD test
**gpsim> Widths 36 0 0 0
module_x 0, module widget size 316, 127
Widths 36 0 0 0
Widths 36 0 0 0
Widths 36 0 36 0
LCD update
Segmentation fault


Regards,
Xiaofan