2021-08-27 16:17:59

by Bryan Brattlof

[permalink] [raw]
Subject: [PATCH 0/5] staging: rtl8723bs: remove unused functions

Changes in v3:
- Fix issue causing some emails to drop their In-Reply-To: header.

(For the curious: NullMailer & my email provider where conspiring)

Changes in v2:
- Remove build warnings I introduced in the previous version.

***

There is a lot of un-used code and un-needed abstractions leftover
in this driver from when (I assume) it could be compiled on more than
one kernel.

This series removes dead code and is one small step toward
refactoring how this driver interacts with the SDIO subsystem.

Bryan Brattlof (5):
staging: rtl8723bs: remove sdio_write_mem()
staging: rtl8723bs: remove sdio_writeN()
staging: rtl8723bs: remove sdio_read_mem() and sdio_readN()
staging: rtl8723bs: remove unused definitions from rtw_io.h
staging: rtl8723bs: remove _read_port from _io_ops structure

drivers/staging/rtl8723bs/hal/sdio_ops.c | 116 ---------------------
drivers/staging/rtl8723bs/include/rtw_io.h | 96 -----------------
2 files changed, 212 deletions(-)

--
2.30.2



2021-08-27 16:24:30

by Bryan Brattlof

[permalink] [raw]
Subject: Re: [PATCH 0/5] staging: rtl8723bs: remove unused functions

Sorry for the noise Greg,

On this day, August 27, 2021 thus sayeth Bryan Brattlof:
> Changes in v3:
> - Fix issue causing some emails to drop their In-Reply-To: header.
>
> (For the curious: NullMailer & my email provider where conspiring)
>

This turns out to be untrue :(

I've been testing my git-send-email setup on some friends without issue.
Somewhere between me and the mailing list In-Reply-To: headers are being
dropped.

I'll be back, but you can safely drop any patches from me today :(

Again sorry for the noise

~Bryan

2021-08-27 16:27:58

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 0/5] staging: rtl8723bs: remove unused functions

On Fri, Aug 27, 2021 at 04:12:56PM +0000, Bryan Brattlof wrote:
> Changes in v3:
> - Fix issue causing some emails to drop their In-Reply-To: header.
>
> (For the curious: NullMailer & my email provider where conspiring)

Did not quite work, see:
https://lore.kernel.org/r/[email protected]
as proof that only this message, and 1/5 were linked together.

The other 4 were out on their own :(

Also, there was no "v3" in the subject line as the documentation asks
for :(

v4 please?

thanks,

greg k-h