On Thu, Jan 22, 2026 at 12:04:42PM +0530, Balaji Selvanathan wrote: > From: Varadarajan Narayanan <[email protected]> > > Do FAT read and write based on the device sector size > instead of the size recorded in the FAT meta data. > > FAT code issues i/o in terms of the sector size. Convert that to > device sector size before doing the actual i/o. Additionally, > handle leading/trailing blocks when the meta data based block > no and i/o size is not an exact multiple of the device sector > size or vice versa. > > Tested on UFS device with sector size 4096 and meta data recorded > sector size 512. > > Signed-off-by: Varadarajan Narayanan <[email protected]> > --- > Changes in v3: > - Respin, no changes > - Link to v2: > https://lore.kernel.org/u-boot/[email protected]/ > > Changes in v2: > - fat_sect_size is declared as static > - The else block in disk_write() function has been removed > - The commit message has additional details about testing > - Link to v1: > https://lore.kernel.org/u-boot/[email protected]/
Have you put this through CI https://docs.u-boot.org/en/latest/develop/ci_testing.html or manually run the filesystem tests locally? It would also be good to know that CI does pass since I saw some divides and modulos in there and knowing we don't run in to the cases where that needs to be a shift or use a helper macro now would be good. Thanks. -- Tom
signature.asc
Description: PGP signature

