Fix Reporting Bugs

This is a work in progress intended for collaboration.

There are also (optional) changes to the pic12 executive included that allow for bus resetting when the gsm_module is enabled.
I must not have committed that last change.  Sorry.  Yeah it passes a 
buffer.  If you mean the get_perf_counter function.  This is an earlier 
version that just had one number for each perf counter.  A later commit 
should have upped it to 5 uint32s and returned it as a buffer.

- tim

On 6/27/14, 6:55 AM, Austin McGee wrote:
>
> In momo_modules/mainboard/src/mib/controller_mib_feature.c:
>
> > @@ -277,6 +286,23 @@ void set_lazy_logging()
> >  	lazy_system_logging = (plist_get_int16(0)==0)?false:true;
> >  }
> >
> > +void read_ram()
> > +{
> > +	unsigned char *val = (unsigned char *)(plist_get_int16(0));
> > +	bus_slave_return_buffer(val, 20);
> > +}
> > +
> > +void get_perf_counter()
> > +{
> > +	uint16 counter = plist_get_int16(0);
> > +	uint32 val = perf_get_counter((PerformanceCounter)counter);
>
> This line causes a compiler warning:
>
> |src/mib/controller_mib_feature.c:298:15: warning: initialization makes integer from pointer without a cast
> |
>
> I think it should actually pass a 20-byte buffer in order to work with 
> the python code, otherwise I don't know how this could work.
>
> Does that seem right? I'll push the changes when I have a chance to 
> test them.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/WellDone/MoMo-Firmware/pull/103/files#r14293330>.
>