mbox series

[0/4] Declare local functions static

Message ID 20240603172311.1587589-1-bvanassche@acm.org (mailing list archive)
Headers show
Series Declare local functions static | expand

Message

Bart Van Assche June 3, 2024, 5:23 p.m. UTC 
Hi Martin,

There are several 32-bit ARM SCSI drivers that trigger compiler warnings about
missing function declarations. This patch series fixes these compiler warnings
by declaring local functions static. Please consider this patch series for the
next merge window.

Thanks,

Bart.

Bart Van Assche (4):
  scsi: acornscsi: Declare local functions static
  scsi: cumana: Declare local functions static
  scsi: eesox: Declare local functions static
  scsi: powertec: Declare local functions static

 drivers/scsi/arm/acornscsi.c | 9 ++++-----
 drivers/scsi/arm/cumana_2.c  | 2 +-
 drivers/scsi/arm/eesox.c     | 2 +-
 drivers/scsi/arm/powertec.c  | 2 +-
 4 files changed, 7 insertions(+), 8 deletions(-)

Comments

Johannes Thumshirn June 3, 2024, 5:51 p.m. UTC | #1 
On 03.06.24 19:23, Bart Van Assche wrote:
> Hi Martin,
> 
> There are several 32-bit ARM SCSI drivers that trigger compiler warnings about
> missing function declarations. This patch series fixes these compiler warnings
> by declaring local functions static. Please consider this patch series for the
> next merge window.
> 
> Thanks,
> 
> Bart.
> 
> Bart Van Assche (4):
>    scsi: acornscsi: Declare local functions static
>    scsi: cumana: Declare local functions static
>    scsi: eesox: Declare local functions static
>    scsi: powertec: Declare local functions static
> 
>   drivers/scsi/arm/acornscsi.c | 9 ++++-----
>   drivers/scsi/arm/cumana_2.c  | 2 +-
>   drivers/scsi/arm/eesox.c     | 2 +-
>   drivers/scsi/arm/powertec.c  | 2 +-
>   4 files changed, 7 insertions(+), 8 deletions(-)
> 
> 
> 

For the series,
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc>
Martin K. Petersen June 5, 2024, 2:23 a.m. UTC | #2 
Bart,

> There are several 32-bit ARM SCSI drivers that trigger compiler
> warnings about missing function declarations. This patch series fixes
> these compiler warnings by declaring local functions static. Please
> consider this patch series for the next merge window.

Applied to 6.11/scsi-staging, thanks!

Martin
Martin K. Petersen June 12, 2024, 1:59 a.m. UTC | #3 
On Mon, 03 Jun 2024 10:23:07 -0700, Bart Van Assche wrote:

> There are several 32-bit ARM SCSI drivers that trigger compiler warnings about
> missing function declarations. This patch series fixes these compiler warnings
> by declaring local functions static. Please consider this patch series for the
> next merge window.
> 
> Thanks,
> 
> [...]

Applied to 6.11/scsi-queue, thanks!

[1/4] scsi: acornscsi: Declare local functions static
      https://git.kernel.org/mkp/scsi/c/f5a954bbf2f4
[2/4] scsi: cumana: Declare local functions static
      https://git.kernel.org/mkp/scsi/c/1414045725a0
[3/4] scsi: eesox: Declare local functions static
      https://git.kernel.org/mkp/scsi/c/1dc98be41814
[4/4] scsi: powertec: Declare local functions static
      https://git.kernel.org/mkp/scsi/c/daf613331c93