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

Dev me12 fthr support #469

Merged
merged 5 commits into from
Mar 16, 2023
Merged

Dev me12 fthr support #469

merged 5 commits into from
Mar 16, 2023

Conversation

karaanil
Copy link
Contributor

@karaanil karaanil commented Mar 6, 2023

No description provided.

@karaanil karaanil requested a review from ozersa March 6, 2023 07:54
if ((error = MXC_UART_Init(WRITING_UART, UART_BAUD, MXC_UART_APB_CLK, MAP_A)) != E_NO_ERROR) {
#else
if ((error = MXC_UART_Init(WRITING_UART, UART_BAUD, MXC_UART_APB_CLK, MAP_B)) != E_NO_ERROR) {
Copy link
Contributor

Choose a reason for hiding this comment

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

#if else condition seems not needed.
The function and parameters are exactly same.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

if calls MAP_A else calls MAB_B. They are not exactly same.

Copy link
Contributor

Choose a reason for hiding this comment

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

The UART0B pins are accessible on the EvKit (RX: P0.7; TX: P0.8). Would it be easier if both the EvKit and FTHR boards use UART0B as the WRITING_UART and UART1A as the READING_UART? Unless you want to show the MAP_A vs MAP_B difference, this will minimize the amount of #if,#else,#endif directives used between the two boards if both run on the same set of pins.

if ((error = MXC_UART_Init(READING_UART, UART_BAUD, MXC_UART_APB_CLK, MAP_A)) != E_NO_ERROR) {
#else
if ((error = MXC_UART_Init(READING_UART, UART_BAUD, MXC_UART_APB_CLK, MAP_A)) != E_NO_ERROR) {
Copy link
Contributor

Choose a reason for hiding this comment

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

#if else condition seems not needed.
The function and parameters are exactly same.
To simplify code it should be deleted.

Copy link
Contributor Author

@karaanil karaanil Mar 7, 2023

Choose a reason for hiding this comment

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

done 5ce180a


#include <stdio.h>

#ifndef LIBRARIES_BOARDS_MAX32662_EVKIT_V1_INCLUDE_BOARD_H_
Copy link
Contributor

Choose a reason for hiding this comment

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

Using FTHR would be better instead of EVKIT_V1

Copy link
Contributor Author

@karaanil karaanil Mar 7, 2023

Choose a reason for hiding this comment

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

done 5ce180a

ARM-DSP/arm_sin_cos_example
ARM-DSP/arm_svm_example
ARM-DSP/arm_variance_example
Bootloader_Host
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove

  • Bootloader_Host
  • EEPROM_Emulator
  • I2C_Manager (if you validated it is ok)

Copy link
Contributor Author

@karaanil karaanil Mar 7, 2023

Choose a reason for hiding this comment

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

done 5ce180a

Note : I2C_Manager is skipped. Please check confluence.

Copy link
Contributor

@ozersa ozersa left a comment

Choose a reason for hiding this comment

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

Comment added

@ozersa ozersa requested a review from sihyung-maxim March 7, 2023 06:24
@karaanil karaanil force-pushed the dev-me12-fthr-support branch from da0dea6 to 1a9774e Compare March 7, 2023 07:52
Copy link
Contributor

@sihyung-maxim sihyung-maxim left a comment

Choose a reason for hiding this comment

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

UART Example:

The UART0B pins are accessible on the EvKit (RX: P0.7; TX: P0.8). Would it be easier if both the EvKit and FTHR boards use UART0B as the WRITING_UART and UART1A as the READING_UART? Unless you want to show the MAP_A vs MAP_B difference, this will minimize the amount of #if,#else,#endif directives used between the two boards if both run on the same set of pins.

@karaanil karaanil force-pushed the dev-me12-fthr-support branch from 1a9774e to b8eee90 Compare March 13, 2023 08:12
@karaanil
Copy link
Contributor Author

karaanil commented Mar 13, 2023

@sihyung-maxim UART example is done 5ce180a

@karaanil karaanil force-pushed the dev-me12-fthr-support branch from b8eee90 to 5ce180a Compare March 13, 2023 08:18
@ozersa
Copy link
Contributor

ozersa commented Mar 16, 2023

@sihyung-maxim could you merge it?

@sihyung-maxim sihyung-maxim merged commit fa96b64 into main Mar 16, 2023
@sihyung-maxim sihyung-maxim deleted the dev-me12-fthr-support branch March 16, 2023 17:35
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.

3 participants