[PATCH 1/1] disk: honor CONFIG_SYS_64BIT_LBA

[Tom Rini]

On Mon, Jul 04, 2022 at 09:30:24AM +0900, AKASHI Takahiro wrote:
> On Sat, Jul 02, 2022 at 02:20:21PM +0200, Heinrich Schuchardt wrote:
> > Without the patch for qemu-x86_defconfig:
> > 
> > * sizeof(lbaint_t) = 4 in dev_read()
> > * sizeof(lbaint_t) = 8 in blkcache_read()
> > 
> > CONFIG_SYS_64BIT_LBA is defined in common.h via
> > include/configs/x86-common.h:22.
> > 
> > We have to include common.h before including blk.h.
> 
> Well, if so, why not include common.h (or config.h?) in blk.h
> since lbaint_t is defined in blk.h which directly depends on some CONFIG_*?
> (This fact is not trivial to developers.)
> 
> Otherwise, the same issue arises somewhere else again.

The complete solution, as found in -next, is to migrate to Kconfig.
This is a stop-gap for the release.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220703/763bd11c/attachment.sig>