On Wed, 8 Nov 2023 at 09:49, Sean Anderson <sean...@gmail.com> wrote: > > Instead of relying on the presence of filename to determine whether we are > dealing with a FAT filesystem (and should DMA-align the buffer), have FAT set > bl_len to ARCH_DMA_MINALIGN instead. With this done, we can remove the > special-case logic checking for the presence of filename. > > Because filesystems are not block-based, we may read less than the size passed > to spl_load_info.read. This can happen if the file size is not DMA-aligned. > This > is fine as long as we read the amount we originally wanted to. Modify the > conditions for callers of spl_load_info.read to check against the original, > unaligned size to avoid failing spuriously. > > Signed-off-by: Sean Anderson <sean...@gmail.com> > --- > > Changes in v7: > - Split off from the removal of filename > > common/spl/spl_blk_fs.c | 3 ++- > common/spl/spl_fat.c | 2 +- > common/spl/spl_fit.c | 23 +---------------------- > common/spl/spl_imx_container.c | 8 +++++--- > 4 files changed, 9 insertions(+), 27 deletions(-) >
Reviewed-by: Simon Glass <s...@chromium.org>
