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

Implement LittleFS for LN882H - only first basic testing from WebApp … #1191

Merged
merged 16 commits into from
May 12, 2024

Commits on May 1, 2024

  1. Implement LittleFS for LN882H - only first basic testing from WebApp …

    …!!!!
    
    - need to add littlefs-source to sdk/OpenLN882H/project/OpenBeken/CMakeLists.txt
    
    Remarks:
    I had to move the import part of LittleFS
    
    in various cmd-source files to nearer to the top of the files
    
    otherwise I had very strange (false!!!) compiler errors regarding
    "conflicting types" for equal(!) typed prototypes and functions or even variables in .h and .c files
    
    e.g.:
    
    /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:47:6: error: conflicting types for 'g_powersave'
       47 | bool g_powersave;
          |      ^~~~~~~~~~~
    In file included from /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_local.h:4,
                     from /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:6:
    /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_public.h:33:13: note: previous declaration of 'g_powersave' was here
       33 | extern bool g_powersave;
          |             ^~~~~~~~~~~
    /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:47:6: error: conflicting types for 'g_powersave'
       47 | bool g_powersave;
          |      ^~~~~~~~~~~
    
    Also I had to disable other drivers, or the OTA image would not be flashed (after rebbot old version was still present)
    Is there a size limit to respect for OTA? the size was well below the shown OTA-size of 0xAA000 (696320k if I calculated correct)
    MaxineMuster committed May 1, 2024
    Configuration menu
    Copy the full SHA
    4799f08 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2024

  1. Moved "bool" from local int definition to stdbool.h

    So I got rid of all those "conflicting types" errors.
    
    Thanks to @giedriuslt for pointing this out!
    
    This commit also includes a define of "ENABLE_TEST_COMMANDS" for the littlefs tests (lfs_test<n>).
    MaxineMuster committed May 2, 2024
    Configuration menu
    Copy the full SHA
    e41eae3 View commit details
    Browse the repository at this point in the history
  2. Restored all #defines to 1

    Last image loaded without any problem
    MaxineMuster committed May 2, 2024
    Configuration menu
    Copy the full SHA
    9e4c7b2 View commit details
    Browse the repository at this point in the history

Commits on May 3, 2024

  1. Update cmd_main.c - temporarily removed one define to be able to sync…

    … to master
    
    Changed 
    #if (defined WINDOWS) || (defined PLATFORM_BEKEN) || (defined PLATFORM_BL602) || (defined PLATFORM_LN882H)
    back to
    #if (defined WINDOWS) || (defined PLATFORM_BEKEN) || (defined PLATFORM_BL602)
    
    in src/cmnds/cmd_main.c
    MaxineMuster committed May 3, 2024
    Configuration menu
    Copy the full SHA
    7821e53 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ee86a43 View commit details
    Browse the repository at this point in the history
  3. Update cmd_main.c

    Restored change in cmd_main.c after syncing
    MaxineMuster committed May 3, 2024
    Configuration menu
    Copy the full SHA
    070f610 View commit details
    Browse the repository at this point in the history
  4. Update user_main.c

    Changed user_main.c to allow SVM_RunThreads() for LN882H with LittleFS
    MaxineMuster committed May 3, 2024
    Configuration menu
    Copy the full SHA
    d830b73 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    0a2feb1 View commit details
    Browse the repository at this point in the history

Commits on May 4, 2024

  1. Configuration menu
    Copy the full SHA
    10f8090 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    bbbbc3c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    0118a3e View commit details
    Browse the repository at this point in the history
  4. Added LN882H to some more "#if defined" lines to allow scripting ...

    Added BL602 in two lines missing this platform, too
    MaxineMuster committed May 4, 2024
    Configuration menu
    Copy the full SHA
    b1631a5 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    dde240e View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    3d639e6 View commit details
    Browse the repository at this point in the history

Commits on May 5, 2024

  1. Configuration menu
    Copy the full SHA
    3bd643d View commit details
    Browse the repository at this point in the history

Commits on May 12, 2024

  1. Configuration menu
    Copy the full SHA
    a62dde3 View commit details
    Browse the repository at this point in the history