lkml.org 
[lkml]   [2021]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 50/50] staging: rtl8723bs: macro DRIVER_PREFIX expands to lowercase driver name
    On Tue, Apr 27, 2021 at 01:04:09PM +0200, Greg KH wrote:
    > On Sat, Apr 24, 2021 at 11:02:33AM +0200, Fabio Aiuto wrote:
    > > macro DRIVER_PREFIX expands to lowercase driver name.
    > >
    > > Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
    > > ---
    > > drivers/staging/rtl8723bs/include/rtw_debug.h | 2 +-
    > > 1 file changed, 1 insertion(+), 1 deletion(-)
    > >
    > > diff --git a/drivers/staging/rtl8723bs/include/rtw_debug.h b/drivers/staging/rtl8723bs/include/rtw_debug.h
    > > index 189f02a8300a..3a5b083e95a1 100644
    > > --- a/drivers/staging/rtl8723bs/include/rtw_debug.h
    > > +++ b/drivers/staging/rtl8723bs/include/rtw_debug.h
    > > @@ -7,7 +7,7 @@
    > > #ifndef __RTW_DEBUG_H__
    > > #define __RTW_DEBUG_H__
    > >
    > > -#define DRIVER_PREFIX "RTL8723BS: "
    > > +#define DRIVER_PREFIX "rtl8723bs: "
    >
    > KBUILD_MODNAME should be used here, in the future.

    ok, got it

    >
    > But really, this shouldn't be needed at all, it's kind of pointless (if
    > it is being used, then odds are the places it is being used is wrong...)
    >
    > thanks,
    >
    > greg k-h

    fabio@agape:~/src/git/kernels/staging$ grep -r DRIVER_PREFIX drivers/staging/rtl8723bs/
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CRsvdPageParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CMediaStatusRptParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CMacIdConfigParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CRssiSettingParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CPwrModeParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c: print_hex_dump_debug(DRIVER_PREFIX ": u1H2CPsTuneParm:", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/hal_com.c: print_hex_dump_debug(DRIVER_PREFIX ": c2h_evt_read(): ", DUMP_PREFIX_NONE,
    drivers/staging/rtl8723bs/hal/hal_com.c: print_hex_dump_debug(DRIVER_PREFIX ": c2h_evt_read(): Command Content:\n",
    drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c: print_hex_dump_debug(DRIVER_PREFIX ": C2HPacketHandler_8723B(): Command Content:\n",
    drivers/staging/rtl8723bs/include/rtw_debug.h:#define DRIVER_PREFIX "rtl8723bs: "

    When I recently changed the old macro printing hex dump I
    decided to keep DRIVER_PREFIX. But I can replace it by
    KBUILD_MODNAME.

    Thank you,

    fabio

    \
     
     \ /
      Last update: 2021-04-27 14:42    [W:3.192 / U:1.484 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site