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

Upstream/rx zephyr support pr1.1 #49

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

duynguyenxa
Copy link
Member

@duynguyenxa duynguyenxa commented Nov 18, 2024

This PR is update for RX hal layer for Zephyr RX MCU porting. The support contains:

  • Basic support for Clock Initialization.
  • Hal layer driver support for GPIO, SCI UART

License of hal is as below:
The software:

  • Can be used for any users.
  • Can be re-distributed by any users. (excluding r_emwin_rx)
  • Can be integrated with specified open source code.
    • Amazon FreeRTOS has been now allowed, Renesas would accept other open source.
  • Use condition is basically MIT License, and details are shown in PDF file in FIT Modules directory.
  • Are generated by SmartConfigurator as well, the generated code (each FIT Modules and Code Generator parts) license is same as previous link.

quytranpzz and others added 9 commits September 25, 2024 07:47
This is initial commit to support Renesas RX with RX driver
package (rdp)

Signed-off-by: Quy Tran <[email protected]>
Initial commit for GPIO driver support on board using RX130 MCUs

Signed-off-by: Phi Tran <[email protected]>
Initial commit for PINCTRL driver support on board using RX130 MCUs

Signed-off-by: Phi Tran <[email protected]>
Initial commit for SCI driver support on board using RX130 MCUs

Signed-off-by: Sang Tran <[email protected]>
This update is to update marco of PLL multiple to adapt the change

Signed-off-by: Tran Van Quy <[email protected]>
Update BSP configurations for Renesas RX

Signed-off-by: Tran Van Quy <[email protected]>
…atus variables

Update CMakelists file to add mcu_locks.c for SOC early intialization

Signed-off-by: Keita Kashima <[email protected]>
Add LICENSE file for Renesas RX HAL

Signed-off-by: Tran Van Quy <[email protected]>
Update CMakeLists file for RX HAL to remove duplicate dirs

Signed-off-by: Tran Van Quy <[email protected]>
Comment on lines +2 to +17
* DISCLAIMER
* This software is supplied by Renesas Electronics Corporation and is only intended for use with Renesas products. No
* other uses are authorized. This software is owned by Renesas Electronics Corporation and is protected under all
* applicable laws, including copyright laws.
* THIS SOFTWARE IS PROVIDED "AS IS" AND RENESAS MAKES NO WARRANTIES REGARDING
* THIS SOFTWARE, WHETHER EXPRESS, IMPLIED OR STATUTORY, INCLUDING BUT NOT LIMITED TO WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. ALL SUCH WARRANTIES ARE EXPRESSLY DISCLAIMED. TO THE MAXIMUM
* EXTENT PERMITTED NOT PROHIBITED BY LAW, NEITHER RENESAS ELECTRONICS CORPORATION NOR ANY OF ITS AFFILIATED COMPANIES
* SHALL BE LIABLE FOR ANY DIRECT, INDIRECT, SPECIAL, INCIDENTAL OR CONSEQUENTIAL DAMAGES FOR ANY REASON RELATED TO THIS
* SOFTWARE, EVEN IF RENESAS OR ITS AFFILIATES HAVE BEEN ADVISED OF THE POSSIBILITY OF SUCH DAMAGES.
* Renesas reserves the right, without notice, to make changes to this software and to discontinue the availability of
* this software. By using this software, you agree to the additional terms and conditions found by accessing the
* following link:
* http://www.renesas.com/disclaimer
*
* Copyright (C) 2017 - 2024 Renesas Electronics Corporation. All rights reserved.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This statement is not compatible with Zephyr's license condition.
So, the BSP can't be included in the Zephyr repository.

Needing to change the license condition.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@soburi , thank you, we are under discussion for changing the license of this software package, hopefully we can resolve this issue soon.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. This means we cannot merge this PR at this time.
Please mark as DNM.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@duynguyenxa Change it to Draft.

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