Skip to content

Conversation

@shuopeng-deng
Copy link

Summary of changes

The fs integration tests requires about 10 little fs blocks.
The qspi device used cypress has erase size of 256KB, with a default size of 2MB, the test only
allocates 8 little fs blocks. The tests need about 10 little fs blocks to
work.
Also Added no-led tags to cypress board that only have 1 LED. The
fs integration tests blinks 2 LEDs.

Fixed issue:
#11451

Impact of changes

Migration actions required

Documentation


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[X] Tests / results supplied as part of this PR

CY8CPROTO_062_4343W_GT_Test_Results.txt


Reviewers

@ARMmbed/team-cypress


The fs integration tests requires about 10 little fs blocks.
The qspi erase size is 256K, with the default size of 2MB, this only
allocates 8 little fs blocks. The tests need 10 little fs blocks to
work.
Also Added no-led tags to cypress board that only have 1 LED. The
fs integration tests blinks 2 LEDs.
@ciarmcom
Copy link
Member

ciarmcom commented Dec 6, 2019

@shuopeng-deng, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@yarbcy
Copy link
Contributor

yarbcy commented Dec 8, 2019

@0xc0170 Please review.

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 9, 2019

CI restarted

@mbed-ci
Copy link

mbed-ci commented Dec 9, 2019

Test run: SUCCESS

Summary: 5 of 5 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 removed the needs: CI label Dec 10, 2019
@0xc0170 0xc0170 merged commit 09a2db0 into ARMmbed:master Dec 10, 2019
@adbridge
Copy link
Contributor

adbridge commented Apr 2, 2020

@Mergifyio backport mbed-os-5.15

@adbridge adbridge changed the title increamed qspi size for cypress integration fs tests increased qspi size for cypress integration fs tests Apr 2, 2020
@mergify
Copy link

mergify bot commented Apr 2, 2020

Command backport mbed-os-5.15: failure

No backport have been created

  • Backport to branch mbed-os-5.15 failed

1 similar comment
@mergify
Copy link

mergify bot commented Apr 2, 2020

Command backport mbed-os-5.15: failure

No backport have been created

  • Backport to branch mbed-os-5.15 failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants