All of lore.kernel.org
 help / color / mirror / Atom feed
From: Deepak R Varma <drv@mailo.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Praveen Kumar <kumarpraveen@linux.microsoft.com>,
	outreachy@lists.linux.dev, Larry.Finger@lwfinger.net,
	phil@philpotter.co.uk, paskripkin@gmail.com,
	linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
	saurabh.truth@gmail.com
Subject: Re: [PATCH v4 11/11] staging: r8188eu: Remove unused macros
Date: Fri, 21 Oct 2022 05:04:37 +0530	[thread overview]
Message-ID: <Y1HbDZbZvdme1hJb@debian-BULLSEYE-live-builder-AMD64> (raw)
In-Reply-To: <Y1I0dcZqMJAzHSm2@kroah.com>

On Fri, Oct 21, 2022 at 07:56:05AM +0200, Greg KH wrote:
> On Fri, Oct 21, 2022 at 11:21:06AM +0530, Praveen Kumar wrote:
> > On 21-10-2022 03:02, Deepak R Varma wrote:
> > > Simple variants of macros PlatformEFIOWrite and PlatformEFIORead are
> > > defined but never used. As they do not appear to be designed for anything
> > > significant, we can remove them to avoid unexpected usage.
> > >
> > > Suggested-by: Julia Lawall <julia.lawall@inria.fr>
> > > Signed-off-by: Deepak R Varma <drv@mailo.com>
> > > ---
> > >
> > > Changes in v4:
> > >    1. Patch newly added to the patch set.
> > >
> > >
> > >  drivers/staging/r8188eu/include/rtw_io.h | 14 --------------
> > >  1 file changed, 14 deletions(-)
> > >
> > > diff --git a/drivers/staging/r8188eu/include/rtw_io.h b/drivers/staging/r8188eu/include/rtw_io.h
> > > index 87fcf6c94ff3..e9744694204b 100644
> > > --- a/drivers/staging/r8188eu/include/rtw_io.h
> > > +++ b/drivers/staging/r8188eu/include/rtw_io.h
> > > @@ -285,18 +285,4 @@ void bus_sync_io(struct io_queue *pio_q);
> > >  u32 _ioreq2rwmem(struct io_queue *pio_q);
> > >  void dev_power_down(struct adapter *Adapter, u8 bpwrup);
> > >
> > > -#define PlatformEFIOWrite1Byte(_a, _b, _c)		\
> > > -	rtw_write8(_a, _b, _c)
> > > -#define PlatformEFIOWrite2Byte(_a, _b, _c)		\
> > > -	rtw_write16(_a, _b, _c)
> > > -#define PlatformEFIOWrite4Byte(_a, _b, _c)		\
> > > -	rtw_write32(_a, _b, _c)
> > > -
> > > -#define PlatformEFIORead1Byte(_a, _b)		\
> >
> > Can the naming be reworked to make more Linux friendly ? something like PLATFORM_EFIO_READ1BYTE or better if there are other suggestions?
> > Rest others as applicable ?
>
> All of these should just be removed entirely and the normal calls here
> made instead.  There is no need for these #defines at all.

Hello Greg,
The current patch scope results in removal of these #defines. Can you please
clarify what you mean by "normal calls here made instead"? Do you mean the
calling function should directly call the wrapped functions OR you are
suggesting converting these #defines to inline functions?

Thank you,
./drv

>
> thanks,
>
> greg k-h
>



  reply	other threads:[~2022-10-21  8:20 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-20 21:26 [PATCH v4 00/11] staging: r8188eu: trivial code cleanup patches Deepak R Varma
2022-10-20 21:26 ` [PATCH v4 01/11] staging: r8188eu: use Linux kernel variable naming convention Deepak R Varma
2022-10-20 21:27 ` [PATCH v4 02/11] staging: r8188eu: reformat long computation lines Deepak R Varma
2022-10-20 21:27 ` [PATCH v4 03/11] staging: r8188eu: remove {} for single statement blocks Deepak R Varma
2022-10-20 21:28 ` [PATCH v4 04/11] staging: r8188eu: use htons macro instead of __constant_htons Deepak R Varma
2022-10-20 21:29 ` [PATCH v4 05/11] staging: r8188eu: correct misspelled words in comments Deepak R Varma
2022-10-20 21:30 ` [PATCH v4 06/11] staging: r8188eu: Add space between function & macro parameters Deepak R Varma
2022-10-20 21:30 ` [PATCH v4 07/11] staging: r8188eu: Associate pointer symbol with parameter name Deepak R Varma
2022-10-20 21:31 ` [PATCH v4 08/11] staging: r8188eu: replace leading spaces by tabs Deepak R Varma
2022-10-21  5:34   ` Praveen Kumar
2022-10-20 23:31     ` Deepak R Varma
2022-10-20 21:31 ` [PATCH v4 09/11] staging: r8188eu: Put '{" on the symbol declaration line Deepak R Varma
2022-10-20 21:32 ` [PATCH v4 10/11] staging: r8188eu: Correct missing or extra space in the statements Deepak R Varma
2022-10-20 21:32 ` [PATCH v4 11/11] staging: r8188eu: Remove unused macros Deepak R Varma
2022-10-21  5:51   ` Praveen Kumar
2022-10-21  5:56     ` Greg KH
2022-10-20 23:34       ` Deepak R Varma [this message]
2022-10-21  8:33         ` Greg KH
2022-10-20 23:50           ` Deepak R Varma
2022-10-21  5:20 ` [PATCH v4 00/11] staging: r8188eu: trivial code cleanup patches Philipp Hortmann
2022-10-22  8:05 ` Greg KH
2022-10-21 16:49   ` Deepak R Varma

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Y1HbDZbZvdme1hJb@debian-BULLSEYE-live-builder-AMD64 \
    --to=drv@mailo.com \
    --cc=Larry.Finger@lwfinger.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=kumarpraveen@linux.microsoft.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-staging@lists.linux.dev \
    --cc=outreachy@lists.linux.dev \
    --cc=paskripkin@gmail.com \
    --cc=phil@philpotter.co.uk \
    --cc=saurabh.truth@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.