jlaitine opened a new pull request, #19020:
URL: https://github.com/apache/nuttx/pull/19020

   
   ## Summary
   
   This is a small size optimization for the mmcsd_sdio driver.
   
   When the CONFIG_MMCSD_MMCSUPPORT is disabled, we can remove the mmc 
partition support, saving ~300+ bytes of flash on a 32-bit Arm target. These 
partitions don't exist on SD cards.
   
   ## Impact
   
   No functional impact, removes "mmcsd_switch" support from SD cards. This 
function only implements the CMD6 MMC switch, not SD "switch function", so it 
is safe to remove on systems without MMC.
   
   ## Testing
   
   Building for stm32f746-ws:nsh
   
   ```
   Before change:
   
   LD: nuttx         
   Memory region         Used Size  Region Size  %age Used
               itcm:          0 GB         1 MB      0.00%
              flash:      233696 B         1 MB     22.29%
               dtcm:          0 GB        64 KB      0.00%
              sram1:       15280 B       240 KB      6.22%
              sram2:          0 GB        16 KB      0.00%
   
   After change:
   
   LD: nuttx                                                                    
   Memory region         Used Size  Region Size  %age Used
               itcm:          0 GB         1 MB      0.00%
              flash:      233376 B         1 MB     22.26%
               dtcm:          0 GB        64 KB      0.00%
              sram1:       15280 B       240 KB      6.22%
              sram2:          0 GB        16 KB      0.00%
   ```
   
   Diff is  320 bytes smaller.
   
   Testing of  stm32f7 (sd card mounted on /fs/microsd):
   ```
   nsh> cd /fs/microsd
   nsh> echo "Hello World" > test.txt
   nsh> cat test.txt
   Hello World
   nsh> 
   
   < reboot >
   
   nsh> cd /fs/microsd
   nsh> cat test.txt
   Hello World
   nsh> 
   ```
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to