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

Attachment: signature.asc
Description: PGP signature

Reply via email to