Add read_file calls with prepared mem buffer.

[abel-msk]

Hello.

I fix compile problems for Arduino, but unfortunately steel cannot test in real application.
And of course add new methods in README.

Copy descriptions here

sd_mmc_card

Newly added methods

Type file_ptr_t universal (IDF/Arduino) pointer to File object. FILE for IDF framework and fs::File for Arduino framework

file_ptr_t open_file(const char path, const char mode);
file_ptr_t open_file(std::string const &path, const char *mode);
path - file path to open
mode - mode to open file ("r" - for read "w" - for write)

Return:  file handler  or NULL if fail

size_t block_read_file(file_ptr_t fl, uint8_t *buf, size_t promise_len);
fl - file handler
buf - pointer to memory area where to read data;
promise_len - max num bytes to read.

return: num of read bytes.  

note: subsequent reads start read form next byte it stop read before

void close_file(file_ptr_t fl);
close file

[n-serrette]

Thanks for your contribution, I will look at it once I have a little time

[n-serrette]

Thanks for your work,
I started looking at your PR, here are my first remarks:

• split your work into two PR :
  • one for the addition of the new reading methods on sd_mmc_card
  • the other for the correction on of the file download on sd_file_server
• remove the .sd_file_server.cpp.swp file which seams to be a remnant of your merge conflict issue
• document the methods you added in sd_mmc_card/README.md
• sd_file_server does not compile with the arduino framework

I will test and review your work further, if you do not consider your PRs ready for review could you mark them as draft

Once again thanks for your work, your contribution is greatly apprecied.

[abel-msk]

Hi. Thanks for answer. I made only one PR when I hadn't started fixing sd_file_server yet. I thought subsequent fixes wouldn't go into the initial request. Unfortunately, I'm not very good at making two different requests from the final version. I'll try. And Yes, I didn't test with Arduino framework, because my other code does not work with Arduino framework . But yes, I'll make small profile for testing and check.

…

On 29 Apr 2025, at 19:27, Serrette Nicolas ***@***.***> wrote: n-serrette left a comment (n-serrette/esphome_sd_card#22) <#22 (comment)> Thanks for your work, I started looking at your PR, here are my first remarks: split your work into two PR : one for the addition of the new reading methods on sd_mmc_card the other for the correction on of the file download on sd_file_server remove the .sd_file_server.cpp.swp file which seams to be a remnant of your merge conflict issue document the methods you added in sd_mmc_card/README.md sd_file_server does not compile with the arduino framework I will test and review your work further, if you do not consider your PRs ready for review could you mark them as draft Once again thanks for your work, your contribution is greatly apprecied. — Reply to this email directly, view it on GitHub <#22 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AD3YOUFDI3CDVL24K5BM4QD236SA5AVCNFSM6AAAAAB3BB74NGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMZZGUZDCNJZHA>. You are receiving this because you authored the thread.