Re: [PATCH 24/31] swim: Remove pointless specifiers

Le 16/07/2026 =C3=A0 12:02, Finn Thain a =C3=A9crit=C2=A0:
> If the compiler made these functions "as fast as possible" that wouldn't
> actually help because they involve slow mechanical operations. Remove
> pointless inline function specifiers.

In fact the idea was not to make them as fast as possible but a avoid the =
jump (to keep all the SWIM=20
operations in the same functional block).

>=20
> Signed-off-by: Finn Thain <[email protected]>
> ---
>   drivers/block/swim.c | 20 ++++++++++----------
>   1 file changed, 10 insertions(+), 10 deletions(-)

Reviewed-by: Laurent Vivier <[email protected]>

>=20
> diff --git a/drivers/block/swim.c b/drivers/block/swim.c
> index dae401e649d7..4679f554fec5 100644
> --- a/drivers/block/swim.c
> +++ b/drivers/block/swim.c
> @@ -222,7 +222,7 @@ extern int swim_read_sector_data(struct swim __iomem=
 *base,
>   				 unsigned char *data);
>  =20
>   static DEFINE_MUTEX(swim_mutex);
> -static inline void set_swim_mode(struct swim __iomem *base, int enable)
> +static void set_swim_mode(struct swim __iomem *base, int enable)
>   {
>   	struct iwm __iomem *iwm_base =3D (struct iwm __iomem *)base;
>   	unsigned long flags;
> @@ -250,7 +250,7 @@ static inline void set_swim_mode(struct swim __iomem=
 *base, int enable)
>   	local_irq_restore(flags);
>   }
>  =20
> -static inline int get_swim_mode(struct swim __iomem *base)
> +static int get_swim_mode(struct swim __iomem *base)
>   {
>   	unsigned long flags;
>  =20
> @@ -324,7 +324,7 @@ static inline bool swim_readbit(struct swim __iomem =
*base, int bit)
>   	ret; \
>   })
>  =20
> -static inline void swim_drive(struct swim __iomem *base,
> +static void swim_drive(struct swim __iomem *base,
>   			      enum drive_location location)
>   {
>   	if (location =3D=3D INTERNAL_DRIVE) {
> @@ -343,8 +343,8 @@ static inline void swim_drive(struct swim __iomem *b=
ase,
>   	udelay(1);
>   }
>  =20
> -static inline void swim_motor(struct swim __iomem *base,
> -			      enum motor_action action)
> +static void swim_motor(struct swim __iomem *base,
> +		       enum motor_action action)
>   {
>   	if (action =3D=3D ON) {
>   		swim_action(base, MOTOR_ON);
> @@ -355,7 +355,7 @@ static inline void swim_motor(struct swim __iomem *b=
ase,
>   	}
>   }
>  =20
> -static inline void swim_eject(struct swim __iomem *base)
> +static void swim_eject(struct swim __iomem *base)
>   {
>   	swim_action(base, EJECT);
>   	swim_readbit_timeout(base, DISK_IN, false, 2000 * 1000);
> @@ -370,14 +370,14 @@ static inline void swim_head(struct swim __iomem *=
base, enum head head)
>   		swim_select(base, READ_DATA_0);
>   }
>  =20
> -static inline int swim_step(struct swim __iomem *base)
> +static int swim_step(struct swim __iomem *base)
>   {
>   	swim_action(base, STEP);
>   	udelay(150);
>   	return swim_readbit_timeout_atomic(base, STEP, false, 20 * 1000);
>   }
>  =20
> -static inline int swim_track00(struct swim __iomem *base)
> +static int swim_track00(struct swim __iomem *base)
>   {
>   	int try;
>  =20
> @@ -397,7 +397,7 @@ static inline int swim_track00(struct swim __iomem *=
base)
>   	return -1;
>   }
>  =20
> -static inline int swim_seek(struct swim __iomem *base, int step)
> +static int swim_seek(struct swim __iomem *base, int step)
>   {
>   	if (step < 0) {
>   		swim_action(base, SEEK_NEGATIVE);
> @@ -422,7 +422,7 @@ static inline int swim_seek(struct swim __iomem *bas=
e, int step)
>   	return 0;
>   }
>  =20
> -static inline int swim_track(struct floppy_state *fs,  int track)
> +static int swim_track(struct floppy_state *fs,  int track)
>   {
>   	struct swim __iomem *base =3D fs->swd->base;
>   	int ret;