Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ports/renesas-ra/boards/VK-RA6M3: Another New Board. #10752

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbedNoobNinja
Copy link
Contributor

@mbedNoobNinja mbedNoobNinja commented Feb 14, 2023

Thanks to the kind help of @TakeoTakahashi2020 & the mentioned intention for a new ports earlier (#10595), this confirms 2-nd board is also uPy ready.

ports/renesas-ra/boards/VK-RA6M3: Adding another new dev board
ports/renesas-ra/boards: Based on new RA6M3 processor
ports/renesas-ra: Part of existing RA family.

Tests.log

@TakeoTakahashi2020
Copy link
Contributor

Thanks for your contribution, @mbedNoobNinja .

image

Could you check this code?
I think 0x100000 should be 0x10000 because 32KB block starts from 0x10000.

@mbedNoobNinja
Copy link
Contributor Author

Nice catch @TakeoTakahashi2020 🙏, thank you for your observation, you are correct! I didn't count the zeros and somehow missed it, but it's fixed now.

@TakeoTakahashi2020
Copy link
Contributor

Hello @mbedNoobNinja -San,

#define MICROPY_HW_BOARD_NAME "VK_RA6M3"

You would be able to use "VK-RA6M3" of the product name for this macro definition as well.

@mbedNoobNinja
Copy link
Contributor Author

mbedNoobNinja commented Feb 21, 2023

Hi @TakeoTakahashi2020 -San,

I set the name you suggested, and now I hope everything is fine.

@github-actions
Copy link

github-actions bot commented Mar 2, 2023

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
        rp2:    -8 -0.002% PICO

@codecov
Copy link

codecov bot commented Jul 28, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (05cb140) 98.39% compared to head (7f2cefa) 98.50%.
Report is 292 commits behind head on master.

❗ Current head 7f2cefa differs from pull request most recent head 232898f. Consider uploading reports for the commit 232898f to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10752      +/-   ##
==========================================
+ Coverage   98.39%   98.50%   +0.10%     
==========================================
  Files         158      155       -3     
  Lines       20941    20540     -401     
==========================================
- Hits        20604    20232     -372     
+ Misses        337      308      -29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mbedNoobNinja
Copy link
Contributor Author

mbedNoobNinja commented Jul 31, 2023

Hi @dpgeorge I rebased the PR. Do you consider it's ready for merge, or it needs more work? This PR includes Ehternet support for M3 in it. In other words: the same as #12047 but for M3.

@mbedNoobNinja
Copy link
Contributor Author

M3 gained LCD support. Now the board can take advantage of micropython-nano-gui lib. There is also touch support inspired from here but the user have to process the dots of its own (i.e. manually call the methods touched() / touches() of the LCD calss in the machine modue & see if the returned points are from the area of interest)

@mbedNoobNinja
Copy link
Contributor Author

M3 gained CAM support. Now the board can take advantage of OV7725. Even more the CAM can now be displayed (or not) on the LCD with the CAM calss methods CAM.show_start(LCD) and CAM.show_stop(). Snapshots are also possible to be saved in the internal FLASH which subsequently can be downloaded on PC with mpremote. Methods are CAM.jpeg('filename') and CAM.yuyv('filename'). There are also a various effects that can be applied black and white, sepia etc.

ports/renesas-ra/boards: New processor RA6M3.
ports/renesas-ra: Integration to the existing RA family.

Signed-off-by: mbedNoobNinja <novoltage@gmail.com>
@projectgus
Copy link
Contributor

This is an automated heads-up that we've just merged a Pull Request
that removes the STATIC macro from MicroPython's C API.

See #13763

A search suggests this PR might apply the STATIC macro to some C code. If it
does, then next time you rebase the PR (or merge from master) then you should
please replace all the STATIC keywords with static.

Although this is an automated message, feel free to @-reply to me directly if
you have any questions about this.

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.

None yet

5 participants