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

fs/littlefs: correct some code format in lfs.h by running checkpatch.sh #461

Merged
merged 1 commit into from
Mar 6, 2020

Conversation

liuguo09
Copy link
Contributor

@liuguo09 liuguo09 commented Mar 6, 2020

Signed-off-by: liuhaitao liuhaitao@xiaomi.com

Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
@liuguo09
Copy link
Contributor Author

liuguo09 commented Mar 6, 2020

one config of arm07 build breaks, it should be fixed to pass check build : (

====================================================================================
Configuration/Tool: sama5d3x-ek/nsh,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL

Configuring...
Copy files
Select CONFIG_HOST_LINUX=y
Refreshing...
Disabling CONFIG_ARMV7A_TOOLCHAIN_GNU_EABIL
Enabling CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
Refreshing...
Building NuttX...

chip/sam_memories.c: In function 'periphb_physregaddr':
chip/sam_memories.c:86:5: warning: "SAM_PERIPHB_PSECTION" is not defined, evaluates to 0 [-Wundef]
86 | #if SAM_PERIPHB_PSECTION != SAM_PERIPHB_VSECTION
| ^~~~~~~~~~~~~~~~~~~~
chip/sam_memories.c:95:10: error: 'SAM_PERIPHB_PSECTION' undeclared (first use in this function); did you mean 'SAM_PERIPHA_PSECTION'?
95 | return SAM_PERIPHB_PSECTION + sectoffset;
| ^~~~~~~~~~~~~~~~~~~~
| SAM_PERIPHA_PSECTION
chip/sam_memories.c:95:10: note: each undeclared identifier is reported only once for each function it appears in
chip/sam_memories.c: In function 'sysc_physregaddr':
chip/sam_memories.c:149:5: warning: "SAM_SYSC_PSECTION" is not defined, evaluates to 0 [-Wundef]
149 | #if SAM_SYSC_PSECTION != SAM_SYSC_VSECTION
| ^~~~~~~~~~~~~~~~~
chip/sam_memories.c:158:10: error: 'SAM_SYSC_PSECTION' undeclared (first use in this function); did you mean 'SAM_SYSC_VSECTION'?

@patacongo patacongo changed the base branch from master to pr461 March 6, 2020 17:52
@patacongo
Copy link
Contributor

one config of arm07 build breaks, it should be fixed to pass check build : (

That is not relevant to this change, is it? That is a SAMA5Dx and has nothing to do with LittleFS. I will continue with the merge.

@patacongo patacongo merged commit e320dec into apache:pr461 Mar 6, 2020
@liuguo09
Copy link
Contributor Author

liuguo09 commented Mar 7, 2020

ok, thanks

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.

None yet

2 participants