lfs_vfs:Use provided lfs_getattr / lfs_setattr instead of lfs_file_getattr / lfs_ file_setattr#14901
lfs_vfs:Use provided lfs_getattr / lfs_setattr instead of lfs_file_getattr / lfs_ file_setattr#14901crafcat7 wants to merge 1 commit intoapache:masterfrom
Conversation
…tattr / lfs_ file_setattr Summary: We can replace lfs_file_xxxattr with lfs_getattr / lfs_setattr already integrated into littlefs Signed-off-by: chenrun1 <chenrun1@xiaomi.com>
|
[Experimental Bot, please feedback here] This PR does not fully meet the NuttX requirements. While it addresses most sections, it has critical omissions and weaknesses:
Specific Recommendations:
By addressing these points, the PR will better meet NuttX requirements and be more likely to be accepted. |
Summary
This change is still under discussion:
#14844
#14479
Before the littlefs community merges lfs_file_getattr / lfs_file_setattr, use the existing lfs_setattr / lfs_getattr to achieve the same purpose.
This change may be relatively cumbersome because we need to convert from filep to path.
Changes
The implementation is as summarized above
Impact
Is a new feature added?: NO
Impact on build: NO
Impact on hardware: NO, this change does not specifically target any particular hardware architectures or boards.
Impact on documentation: NO,This patch does not introduce any new features
Impact on compatibility: This implementation aims to be backward compatible. No existing functionality is expected to be broken.
Testing
Build Host(s): Linux x86
Target(s): sim/nsh
It is insensitive to the use of the upper layer. We use another interface to implement the equivalent method.