Re: [PATCH 16/30] ide: Avoid preprocessor for CONFIG_LBA48

2023-04-18 Thread Simon Glass
Hi Mattijs, On Tue, 28 Mar 2023 at 08:18, Mattijs Korpershoek wrote: > > On mar., mars 28, 2023 at 08:07, Simon Glass wrote: > > > Use IS_ENABLED() instead for all conditions. Add the 'lba48' flag into > > struct blk_desc always, since it uses very little space. Use a bool so > > the meaning is

Re: [PATCH 16/30] ide: Avoid preprocessor for CONFIG_LBA48

2023-03-28 Thread Mattijs Korpershoek
On mar., mars 28, 2023 at 08:07, Simon Glass wrote: > Use IS_ENABLED() instead for all conditions. Add the 'lba48' flag into > struct blk_desc always, since it uses very little space. Use a bool so > the meaning is clearer. > > Signed-off-by: Simon Glass > --- > > drivers/block/ide.c | 57 +

[PATCH 16/30] ide: Avoid preprocessor for CONFIG_LBA48

2023-03-27 Thread Simon Glass
Use IS_ENABLED() instead for all conditions. Add the 'lba48' flag into struct blk_desc always, since it uses very little space. Use a bool so the meaning is clearer. Signed-off-by: Simon Glass --- drivers/block/ide.c | 57 - include/blk.h | 4 +