lkml.org 
[lkml]   [2021]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/3] staging: r8188eu: Makefile: don't overwrite global settings
From
On 10/16/21 19:35, Martin Kaiser wrote:
> Remove settings from the Makefile that are set by higher-level Makefiles.
>
> Some of those settings might have been useful when the driver was
> maintained out of tree.
>
> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
> drivers/staging/r8188eu/Makefile | 14 --------------
> 1 file changed, 14 deletions(-)
>
> diff --git a/drivers/staging/r8188eu/Makefile b/drivers/staging/r8188eu/Makefile
> index fccf7e6d1520..8294fb69ecf9 100644
> --- a/drivers/staging/r8188eu/Makefile
> +++ b/drivers/staging/r8188eu/Makefile
> @@ -1,8 +1,3 @@
> -SHELL := /bin/bash
> -EXTRA_CFLAGS += $(USER_EXTRA_CFLAGS)
> -EXTRA_CFLAGS += -O1
> -
> -ccflags-y += -D__CHECK_ENDIAN__
>
> OUTSRC_FILES := \
> hal/HalHWImg8188E_MAC.o \
> @@ -46,15 +41,6 @@ _OS_INTFS_FILES := \
>
> _HAL_INTFS_FILES += $(OUTSRC_FILES)
>
> -SUBARCH := $(shell uname -m | sed -e "s/i.86/i386/; s/ppc.*/powerpc/; s/armv.l/arm/; s/aarch64/arm64/;")
> -
> -ARCH ?= $(SUBARCH)
> -CROSS_COMPILE ?=
> -KVER ?= $(if $(KERNELRELEASE),$(KERNELRELEASE),$(shell uname -r))
> -KSRC ?= $(if $(KERNEL_SRC),$(KERNEL_SRC),/lib/modules/$(KVER)/build)
> -MODDESTDIR := /lib/modules/$(KVER)/kernel/drivers/net/wireless
> -INSTALL_PREFIX :=
> -
> rtk_core := \
> core/rtw_ap.o \
> core/rtw_br_ext.o \
>

Hi Martin, this patch introduces a GCC warning on my system.

drivers/staging/r8188eu/os_dep/ioctl_linux.c: In function
'rtw_wx_set_enc_ext':
drivers/staging/r8188eu/os_dep/ioctl_linux.c:1929:9: warning: 'strncpy'
specified bound 16 equals destination size [-Wstringop-truncation]
1929 | strncpy((char *)param->u.crypt.alg, alg_name,
IEEE_CRYPT_ALG_NAME_LEN);
|
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Regards,
Michael

\
 
 \ /
  Last update: 2021-10-17 14:18    [W:0.647 / U:1.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site