Skip to content

Conversation

deepikabhavnani
Copy link

@deepikabhavnani deepikabhavnani commented Oct 17, 2017

This PR is to support IAR8 and verify across all targets and modules.

Implementation

  1. Dynamic Heap Support in IAR - Add dynamic heap support to IAR #4955
  2. Multiple Targets does not build with IAR8.2 #5380

@deepikabhavnani
Copy link
Author

@studavekar could we have a test run using IAR 8.2 on feature-iar8 branch

@studavekar
Copy link
Contributor

@deepikabhavnani we would plan to have testing on 8.2 in few weeks from now.

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

Build : FAILURE

Build number : 199
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5329/

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 19, 2017

@deepikabhavnani we would plan to have testing on 8.2 in few weeks from now.

This means that this is blocked until 8.x is in CI, testing required

@studavekar
Copy link
Contributor

studavekar commented Oct 19, 2017

Internal tracking ticket IOTTESTINF-693 , i guess need version of IAR 8.2 and above.

@adbridge
Copy link
Contributor

@deepikabhavnani Also looks like this now needs a rebase..

Two new block properties available in linker configuration files:
"expanding size" and "minimum size" for IAR 8.11.2 and above

Sample:
define block HEAP with expanding size, minimum size = 16K, alignment = 8 {};

This block will expand to consume all remaining available space in
the range where it is placed. If several such blocks end up in the same
range, they will share the remaining space.
Cannot place a block with expanding size inside another block with
expanding size, inside a block with maximum size, or inside an overlay.
@deepikabhavnani
Copy link
Author

Rebased to resolve conflicts

@deepikabhavnani
Copy link
Author

"UBLOX_EVK_ODIN_W2" target build fails with IAR8.2

[Error] stm32f4xx_hal_def.h@132,30: [Pe167]: argument of type "unsigned long volatile *" is incompatible with parameter of type "unsigned int const volatile *"
        [Error] stm32f4xx_hal_def.h@134,34: [Pe167]: argument of type "unsigned long volatile *" is incompatible with parameter of type "unsigned int volatile *"
        [Error] stm32f4xx_hal_def.h@142,30: [Pe167]: argument of type "unsigned long volatile *" is incompatible with parameter of type "unsigned int const volatile *"
        [Error] stm32f4xx_hal_def.h@144,0: [Pe167]: argument of type "unsigned long volatile *" is incompatible with parameter of type "unsigned int volatile *"

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 25, 2017

"UBLOX_EVK_ODIN_W2" target build fails with IAR8.2

Are these errors coming from a driver, as the error indicates wrong use - incorrect parameters for some functions. And why suddenly IAR 8.x shows these? Is IAR 8.x binary compatible (we can use prebuilt binaries from 7.x) ?

@deepikabhavnani
Copy link
Author

deepikabhavnani commented Oct 25, 2017

IAR8.x is not binary compatible with 7.x. I am verifying dynamic heap with IAR8.2 on linux machine with few targets.
Realtek Ameba and ODIN boards were previously excluded from IAR8.x builds because of incompatible libraries.

@mbed-ci
Copy link

mbed-ci commented Oct 26, 2017

Build : FAILURE

Build number : 341
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5329/

@studavekar
Copy link
Contributor

@deepikabhavnani now have CI feature-iar8 , all PR from this branch will be tested against IAR 8.2 #5329 (comment)

@studavekar
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 26, 2017

Build : FAILURE

Build number : 344
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5329/

@deepikabhavnani
Copy link
Author

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2017

Build : FAILURE

Build number : 390
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5329/

@adbridge
Copy link
Contributor

@deepikabhavnani Looks like lots of IAR failures...

RAM define/section if splited, IAR exanpding size considers just one
section while heap creation which results in smaller chunk for heap.
This commit is to unify the RAM section.
@deepikabhavnani
Copy link
Author

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2017

Build : FAILURE

Build number : 394
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5329/

@deepikabhavnani
Copy link
Author

/morph test

@studavekar
Copy link
Contributor

@deepikabhavnani awesome just ublox failure 👍

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2017

@deepikabhavnani
Copy link
Author

@studavekar - Test failed for UBLOX IAR only. Should we be testing more targets with this feature branch?

@studavekar
Copy link
Contributor

@deepikabhavnani we are testing on all available devices.

@deepikabhavnani
Copy link
Author

IAR8 support moved to 5.8, hence closing this PR

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.

6 participants