On 08/14/2018 04:35 AM, Andreas Dannenberg wrote:
> In order to make the debug print in file_fat_read_at() a tad more useful,
> show the offset the file is being read at alongside the filename.
>
> Suggested-by: Tero Kristo <t-kri...@ti.com>
> Signed-off-by: Andreas Dannenberg <dannenb...@ti.com>
> ---
>
> Small addition but helpful nevertheless as none of the other debug prints
> embedded into fat.c seems to output this info. Without that addition it
> would just tell you the same file name a couple of times when reading
> from an DTB/ITB file for example. With the offset being shown it's easier
> to correlate/debug the loading process.
>
> Oddly (and I haven't fully debugged this) but in order to really get _any_ of
> the debug() prints working in fat.c in my particulat setup, in addition to
> the usual '#define DEBUG' I also had to explicitly define _DEBUG to '1',
> something that should already be taken care of by log.c...
>
>
> fs/fat/fat.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/fat/fat.c b/fs/fat/fat.c
> index 4efe8a3eda..4b722fc5ca 100644
> --- a/fs/fat/fat.c
> +++ b/fs/fat/fat.c
> @@ -1095,7 +1095,7 @@ int file_fat_read_at(const char *filename, loff_t pos,
> void *buffer,
> if (ret)
> goto out_free_both;
>
> - debug("reading %s\n", filename);
> + debug("reading %s at pos %llu\n", filename, pos);
> ret = get_contents(&fsdata, itr->dent, pos, buffer, maxsize, actread);
>
> out_free_both:
>
This seems to duplicate
[PATCH 1/1] fat: provide position in debug message
https://lists.denx.de/pipermail/u-boot/2018-August/337850.html
The only difference is that you print a decimal number instead of a hex
number.
Using %llu is consistent with
fs/fat/fat.c:331:
debug("Read position past EOF: %llu\n", pos);
So I will withdraw my patch.
Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
