Re: [PR] drivers/mtd/ftl/bch: Add direct writing method for FTL [nuttx]

[via GitHub]

fdcavalcanti commented on PR #16642:
URL: https://github.com/apache/nuttx/pull/16642#issuecomment-3089684823

   > > > Hi @xiaoxiang781216 there is a CI build error in xtensa-01 for 
`esp32-devkitc/nxlooper` ，so I tried to do build in my own environment:
   > > > ```
   > > > 1. ./tools/refresh.sh --silent esp32-devkitc/nxloop
   > > > 
   > > > 2. make
   > > > ```
   > > > 
   > > > 
   > > >     
   > > >       
   > > >     
   > > > 
   > > >       
   > > >     
   > > > 
   > > >     
   > > >   
   > > > but another problem happened and I can't finish the build **(even 
without my patch):** maybe this is for a wrong version of xtensa gcc <img 
alt="image" width="1481" height="795" 
src="https://private-user-images.githubusercontent.com/138179549/466559775-7091e4e6-e6b2-4cca-abac-8bcb28cb62a7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTI3NTUzNzcsIm5iZiI6MTc1Mjc1NTA3NywicGF0aCI6Ii8xMzgxNzk1NDkvNDY2NTU5Nzc1LTcwOTFlNGU2LWU2YjItNGNjYS1hYmFjLThiY2IyOGNiNjJhNy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwNzE3JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDcxN1QxMjI0MzdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03ZGI1N2RiZWVmNDAxMTg3YzU1ZjhiODEyODE1NjkzODc5MWY0NGFjYjZiMmMyOGNhMWUyMjE1ZTRkYjE2ZDkxJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.cCAYIBkO1MO6jshEhvkYOn6v3PMV5KZBltHT42duj0M";>
   > > > does anybody konw why this is happening in xtensa01: <img alt="image" 
width="839" height="717" 
src="https://private-user-images.githubusercontent.com/138179549/466566634-0cdc77d9-6224-44ce-89e7-eac6b01c370d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTI3NTUzNzcsIm5iZiI6MTc1Mjc1NTA3NywicGF0aCI6Ii8xMzgxNzk1NDkvNDY2NTY2NjM0LTBjZGM3N2Q5LTYyMjQtNDRjZS04OWU3LWVhYzZiMDFjMzcwZC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwNzE3JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDcxN1QxMjI0MzdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03NDc3MzMxNmY1NWI1ZDgyNTkzOTQyYTUzNDhkNGE3YTI1NmE4NGI1YzY0M2Y4ZmRmMTk4NjE1NDRjNGNkMmZmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.TfFe2JuPkFGW5JlN5MK1tAWeRj3rVrYYbJDLq-5Lf0U";>
   > > 
   > > 
   > > Hi all, seems we still have this issue. I have run a git bisect since 
this CI error is still happening. Looks like it was indeed introduced by 
`drivers/fs: Control the behavior of FTL by passing oflags during the open 
process.`. I'll be taking a look into it, but if you guys have any idea let me 
know.
   > 
   > Thanks for the helping, any process please let me know~
   
   So the issue is not related to this MR specifically. Its an alignment issue 
on ESP32 and it was a coincidence that happened on this PR. Upgrading to 
esptool v5.0.x fixes the problem and we should keep discussing on #16723. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org