Skip to content

Conversation

@javakys
Copy link
Owner

@javakys javakys commented Apr 9, 2021

,

javakys and others added 30 commits July 25, 2018 18:05
fixed unused variables warning
wizchip_init()  - modify about MAX TX/RX buffer when W5100S use.
getSn_RXMEM_SIZE
setSn_TXMEM_SIZE
getSn_TXMEM_SIZE

When that function use, it is used to same with W5100
 - TMSR /RMSR of Common register
fixed -Wsizeof-pointer-memaccess warning
fixed several warnings in mqtt_interface
…g_fix

cleaned up unused variable warning for _WIZCHIP_ >= W5200
cleaned up maybe-unitialized warning
removed unused variable strindex
midnightcow and others added 24 commits February 7, 2019 11:27
1.For W5100S,The array size is one byte smaller
fix w5100s bit define bugs
Fixed an issue where ping ip address failed after pinging domain name failed.
 - Sn_PROTO
Modify getSn_FRAGR,setSn_FRAGR
Modify the bit of IR register definition
Modify getxxx function 
 - setSn_PROTOR , getSn_FRAGR, setSn_FRAGR
 - Modify the bit of IR register definition
QoS value was an enum and this value was passed to MQTTSerialize_subscribe as an (int*).
So when -Os optimization is used the QoS enum was set to byte and placed at odd address.
But when the code reached MQTTSerialize_subscribe and the address of pointer to byte was casted to pointer to integer the LDR instruction caused HardFaul because of unaligned int access.

Fixed by forcing to pass char so now it should not matter which compiler is used.
modify comparing DHCP server ip addres
Fix MQTTClient unaligned access on ARM when using -Os
Fix regression KEEPALIVEAUTO Not supported in W5100, W5200
Compiler warned that there was no break at the end of the case statement. Added the break.
Added a break to remove compiler warning.
It is not compatable to W5100.
It is only valid on W5500
Modify wizchip_clrinterrupt() to set Sn_IR for clearing SIR.
@javakys javakys merged commit 736cc3f into javakys:master Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.