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

Add user apps Kconfig for settings #121

Merged
merged 2 commits into from
Jan 1, 2020
Merged

Add user apps Kconfig for settings #121

merged 2 commits into from
Jan 1, 2020

Conversation

louisom
Copy link
Contributor

@louisom louisom commented Oct 2, 2016

In this moment, build system will build all program in
folder user/apps, but this may cause unexpected effect,
for example, let kdb unusable in UART.

So, we can add a new kconfig file for apps like test cases
or real user program like pingpong, and thus rename menu
title from "Test Case" to "User Space".

And add submenu "Apps" and "Test Cases" for now.

The structure look like this:

  • User Space
    • Compile with user apps
      • Apps
      • Test Cases

@louisom
Copy link
Contributor Author

louisom commented Oct 2, 2016

It seems that travis CI broke by new settings....

$ build/host/conf --silentoldconfig build/host/Config.in
*
* Restart config...
*
*
* User Space
*
Compile with user apps (USER_APPS_INCLUDE) [N/y/?] (NEW) 

@jserv
Copy link
Member

jserv commented Oct 2, 2016

Cc. @georgekang

@jserv
Copy link
Member

jserv commented Oct 2, 2016

user/apps is not a folder. It is a "directory" instead.

bool "EXTI interrupt test case"
menu "User Space"

config USER_APPS_INCLUDE
Copy link
Member

Choose a reason for hiding this comment

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

I dislike the use of USER_APPS_INCLUDE. The mnemonic alternative can be USER_APPS_ENABLED.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is true, I'll change it to USER_APPS_ENABLED

Copy link
Member

Choose a reason for hiding this comment

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

Hint: use git rebase -i to squash the changes.

@jserv
Copy link
Member

jserv commented Oct 2, 2016

@grapherd, You have to update Travis-CI configurations right after new options are introduced.

@louisom
Copy link
Contributor Author

louisom commented Oct 2, 2016

I found that there have still some problem need to be clear.

I can only success to see kdb when first make defconfig then disable USER_APPS_ENABLED.

If I first disable USER_APPS_ENABLED then make flash, it won't appear kdb, but only == on screen.

@jserv
Copy link
Member

jserv commented Oct 6, 2016

I expect there are only two commis: one is Kconfig changes, and another is Travis-CI specific update.

@louisom
Copy link
Contributor Author

louisom commented Oct 6, 2016

@jserv fixed, commit change to Add user apps kconfig and update board defconfig

menu "User Space"

config USER_APPS_ENABLED
bool "enabled user apps for f9-kernel"
Copy link
Member

Choose a reason for hiding this comment

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

Simply. Use "build user applications".

Louie Lu added 2 commits January 30, 2017 21:23
In this moment, build system will build all program in
folder user/apps, but this may cause unexpected effect,
for example, let kdb unusable in UART.

So, we can add a new kconfig file for apps like test cases
or real user program like pingpong, and thus rename menu
title from "Test Case" to "User Space".

And add submenu "Apps" and "Test Cases" for now.

The structure look like this:

    - User Space
        - [ ] enabled user apps for f9-kernel
                - Apps
                - Test Cases
@jserv jserv merged commit 0ee9ff7 into f9micro:master Jan 1, 2020
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.

2 participants