[PATCH v3 0/2] get rid of confusing macros

Nam Cao posted 2 patches 3 years, 10 months ago
.../r8188eu/hal/Hal8188ERateAdaptive.c        | 12 +++----
.../r8188eu/include/Hal8188ERateAdaptive.h    | 13 --------
drivers/staging/r8188eu/include/basic_types.h | 31 -------------------
3 files changed, 6 insertions(+), 50 deletions(-)
[PATCH v3 0/2] get rid of confusing macros
Posted by Nam Cao 3 years, 10 months ago
Replace some confusing macros with more explicit codes.

V3:
  - Get rid of GET_TX_REPORT_TYPE1_RERTY_0 and similar
V2:
  - Split into smaller commits so that it's easier to review

Nam Cao (2):
  staging: r8188eu: replace confusing macros
  staging: r8188eu: remove unused macros

 .../r8188eu/hal/Hal8188ERateAdaptive.c        | 12 +++----
 .../r8188eu/include/Hal8188ERateAdaptive.h    | 13 --------
 drivers/staging/r8188eu/include/basic_types.h | 31 -------------------
 3 files changed, 6 insertions(+), 50 deletions(-)

-- 
2.25.1
Re: [PATCH v3 0/2] get rid of confusing macros
Posted by Dan Carpenter 3 years, 10 months ago
On Tue, Jun 14, 2022 at 05:58:43PM +0200, Nam Cao wrote:
> Replace some confusing macros with more explicit codes.
> 
> V3:
>   - Get rid of GET_TX_REPORT_TYPE1_RERTY_0 and similar
> V2:
>   - Split into smaller commits so that it's easier to review
> 
> Nam Cao (2):
>   staging: r8188eu: replace confusing macros
>   staging: r8188eu: remove unused macros

So much better!  Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>

regards,
dan carpenter