[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZyjdzJujaHJekv8Q@gallifrey>
Date: Mon, 4 Nov 2024 14:44:28 +0000
From: "Dr. David Alan Gilbert" <linux@...blig.org>
To: Kalle Valo <kvalo@...nel.org>
Cc: pkshih@...ltek.com, linux-wireless@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] rtlwifi: Remove more unused functions
* Kalle Valo (kvalo@...nel.org) wrote:
> "Dr. David Alan Gilbert" <linux@...blig.org> writes:
>
> > * Kalle Valo (kvalo@...nel.org) wrote:
> >> linux@...blig.org writes:
> >>
> >> > From: "Dr. David Alan Gilbert" <linux@...blig.org>
> >> >
> >> > exhalbtc_dbg_control(), exhalbtc_stack_update_profile_info(),
> >> > exhalbtc_set_hci_version(), and exhalbtc_set_bt_patch_version() are
> >> > unused since their addition in 2014 by
> >> > commit aa45a673b291 ("rtlwifi: btcoexist: Add new mini driver")
> >> >
> >> > Remove them.
> >> >
> >> > Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>
> >>
> >> 'wifi:' missing.
> >
> > Oops, can add that.
> >
> >> Also in both patches the subject could be more unique.
> >
> > Do you have suggestions? I don't know the code to give good titles.
> > They're both 'btcoexist'; I could merge them into one
> >
> > wifi: wtlwifi: Remove btcoexist deadcode
>
> No good suggestions really. Usually I add what I'm removing to the
> subject but as this patch is removing several functions the subject
> would be too long. Maybe something like 'wifi: rtlwifi: remove several
> unused exhalbtc_*() functions?' just to throw out some ideas, I'm sure
> you can come with better ones.
OK, I've just merged the two together, and sent it out as a single patch
[PATCH v2] wifi: rtlwifi: Remove some exhalbtc deadcod
message-id: 20241104144331.29262-1-linux@...blig.org
Dave
>
> --
> https://patchwork.kernel.org/project/linux-wireless/list/
>
> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
>
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
Powered by blists - more mailing lists