mbed-os-5.15: Nuvoton: Fix NuMaker I2C timeout

[cyliangtw]

Summary of changes

This PR is to fix I2C timeout measurement for NuMaker platforms instead of hardcode max timer count on OS-5.15 branch.
The same commit got approval in PR #13679 on OS-6.x master branch.

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)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

---

Reviewers

---

@0xc0170
@andypowers

[ciarmcom]

@cyliangtw, thank you for your changes.
@andypowers @0xc0170 @ARMmbed/mbed-os-maintainers please review.

[andypowers]

Approved.

[0xc0170]

CI started

[0xc0170]

CI restarted

[0xc0170]

@cyliangtw Can you review line endings for files in this PR:

targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c
targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c

Please fix

[cyliangtw]

@cyliangtw Can you review line endings for files in this PR:

targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c
targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c

Please fix

@0xc0170 , Hi
I compared with other platform i2c_api.c, such like as /TARGET_M480/i2c_api.c and TARGET_M251/i2c_api.c . All of the line endings are the same as 0x0d 0x0a. Even, checked TARGET_M480/gpio_api.c, the line endings are still the same.
I can't open the details of continuous-integration/jenkins/pr-head.
So far, I don't know what's the problem on /TARGET_NANO100/i2c_api.c .
More, I just built NANO130 blinky sample with i2c manipulation, it's successful.
How could I reproduce the error of jenkins checking ?

[0xc0170]

If I follow the steps here on Github, I am on 5.15 branch:

git checkout -b OpenNuvoton-nvt_i2c_timeout_5.15 mbed-os-5.15
git pull https://github.com/OpenNuvoton/mbed.git nvt_i2c_timeout_5.15

git status

On branch OpenNuvoton-nvt_i2c_timeout_5.15
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   targets/TARGET_NUVOTON/TARGET_M451/i2c_api.c
        modified:   targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c

no changes added to commit (use "git add" and/or "git commit -a")

git diff:

warning: CRLF will be replaced by LF in targets/TARGET_NUVOTON/TARGET_M451/i2c_api.c.
The file will have its original line endings in your working directory
warning: CRLF will be replaced by LF in targets/TARGET_NUVOTON/TARGET_NANO100/i2c_api.c.
The file will have its original line endings in your working directory
diff --git a/targets/TARGET_NUVOTON/TARGET_M451/i2c_api.c b/targets/TARGET_NUVOTON/TARGET_M451/i2c_api.c

There are line endings errors.

[0xc0170]

Would git add --renormalize fix both files?

[0xc0170]

If executing above command, it should be simple to just amend the previous comments to fix it. I would like to start CI asap to get this in for the next release (early next week).

Pull request has been modified.

[cyliangtw]

@0xc0170 , Thanks a lot of your recommendation.
I fixed the line ending issue at commit 3d0be7d and also could pass the merge steps locally.

[0xc0170]

CI started

[mbed-ci]

Test run: SUCCESS

Summary: 10 of 10 test jobs passed
Build number : 7
Build artifacts