Skip to content

Conversation

@namanjain7
Copy link
Contributor

No description provided.

jylee9613 and others added 3 commits December 24, 2025 11:39
Current test code considers about kernel/common/app1 partition, not about app2 partition.
This commit has been modified to enable testing even when app2 partition present.

Signed-off-by: Jaeyong Lee <jaeyong1.lee@samsung.com>
…rnel update test to accept command line arguments

- Add support for running specific tests: same_version, new_version, invalid, all
- Each test now runs independently with proper error handling
- Update main function to parse argc/argv and call appropriate test functions
… update tests separately

This commit modifies the wifiapp to allow users to run individual binary update tests
instead of only running all tests together
@namanjain7
Copy link
Contributor Author

#7086 #7087 #7095
PR raised on BK_728N_Support branch

@amandeep-samsung
Copy link
Contributor

This is for OTA test for beken chipset and includes changes done by [jylee9613] for app2 binary support in test.
Other changes are done for having options of running all tests independently.

These are verified on both chipsets; bk7239n and rtl8721csm.

@seokhun-eom24
Copy link
Contributor

Can you test this example is also verified for rtl8730e?

@namanjain7
Copy link
Contributor Author

Can you test this example is also verified for rtl8730e?

verified on ailite board.

Copy link
Contributor

@jylee9613 jylee9613 left a comment

Choose a reason for hiding this comment

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

looks good to me.

…sion test

Earlier, teh new_version test shows status as failed because version_up flag was set as false. This flag should be true
@namanjain7
Copy link
Contributor Author

Added new commit for kernel_update test for new_version.
Version_up was disabled earlier. setting the flag is true so that test do not show fail status even if it's passed.

if (bin_names[i] == KERNEL) {
version_up = true;
}
int version_up = true;
Copy link
Contributor

Choose a reason for hiding this comment

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

I set version_up to true only for kernel, because this is kernel update test.
What I intended is that even if only kernel binary is updated, binary manager should load the latest version of kernel and same version of app1/app2.
app1/app2 version update test need to be tested in loadable_sample/wifiapp/binary_update.c

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.

5 participants