Show Posts

This section allows you to view all posts made by this member. Note that you can only see posts made in areas you currently have access to.


Messages - colinbathe

Pages: [1]
1
STTM STM32 and STR91XF / Re: Ethernet Pins on STM32F1xx
« on: December 14, 2012, 06:43:23 PM »
Yep my mistake. Wasn't reading the code correctly.

<goes off to get his eyes checked...>

2
STTM STM32 and STR91XF / Ethernet Pins on STM32F1xx
« on: December 14, 2012, 04:07:25 PM »

V1.4.1 stm32.c

fnConfigEthernet()


This code sets up the ethernet pins for STM32F1xx processors using the alternative pin set. However, the processor isn't told to remap the ethernet port pins. _PERIPHERAL_REMAP(ETH_REMAP); sorts this.

(probably sorts this - I don't have ethernet working yet...)

3
STTM STM32 and STR91XF / Re: STM32 - IAR Workbench build errors
« on: December 13, 2012, 03:03:53 PM »
For 2) I should have posted more of the code.

It is basically

/*


/*****/

which it is correctly complaining about.

It should be either

/*



*****/


or better

/*



*/
/*****/

(this is probably one of the most exciting bugs you have had to deal with....  ::) )

4
STTM STM32 and STR91XF / STM32 - IAR Workbench build errors
« on: December 12, 2012, 06:18:01 PM »
Hi,

I've recently been playing with uTasker on STM32F107 on IAR workbench 6.5 and it compiles with the following warnings:

(line numbers may be a little out as I've been playing with the code)

STM32.c 
Warning[Pe111]: statement is unreachable C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 322


#endif
    return 0;                                                            // we never return....
}
#endif

Expected error - no action required.

Warning[Pe009]: nested comment is not allowed C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 2001

/*********************************************/

Action - remove leading "/"

Warning[Pa082]: undefined behavior: the order of volatile accesses is undefined in this statement C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 633

__interrupt void EMAC_Interrupt(void)
{
    while ((ETH_DMAIER & ETH_DMASR) & (ETH_DMAIER_TIE | ETH_DMAIER_TPSIE | ETH_DMAIER_TBUIE | ETH_DMAIER_TJTIE |
                                       ETH_DMAIER_ROTIE | ETH_DMAIER_TUIE | ETH_DMAIER_RIE | ETH_DMAIER_RBUIE |
                                       ETH_DMAIER_RPSIE | ETH_DMAIER_RWTIE | ETH_DMAIER_ETIE | ETH_DMAIER_FBEIE |
                                       ETH_DMAIER_ERIE | ETH_DMAIER_AISE | ETH_DMAIER_NISE)) { // while enabled EMAC interrupt pending

Action: Not sure if this is an issue or not.


http.c 
Warning[Pe546]: transfer of control bypasses initialization of: C:\Google Drive\Projects\ST100\uTasker\stack\http.c 994
            variable "Len" (declared at line 1111 of "C:\Google Drive\Projects\ST100\uTasker\stack\http.c")
            variable "ucPush" (declared at line 1112 of "C:\Google Drive\Projects\ST100\uTasker\stack\http.c")


    for (iSessionNumber = 0; iSessionNumber < NO_OF_HTTP_SESSIONS; iSessionNumber++) {
        if (http_session->OwnerTCPSocket == Socket) {                    // search for the session to handle this event
            switch (ucEvent) {
            case TCP_EVENT_CONREQ:                                       // session requested
                if (http_session->ucState == HTTP_STATE_FREE) {

...

                if (http_session->FileLength > http_session->FilePoint) {
                    MAX_FILE_LENGTH Len = (http_session->FileLength - http_session->FilePoint);
                    unsigned char ucPush = 0;

Action: Not sure if this is a problem or not.


I hope this was useful.

Colin

Pages: [1]