On 8/1/23 21:27, Brian Norris wrote:
I'm not sure that's truly an appropriate "Fixes" target, as that commit just shuffled the existing bad code around. I'd either remove that line, or else also add: Fixes: d930faee141b ("mwifiex: add support for Marvell pcie8766 chipset") where the buggy stuff was first introduced. I don't think you need to resubmit just for this; Kalle can usually make small adjustments like this when applying. And even if not, the issue is trivial.
I have some more PCIe bits; would it be better to submit them separately or create a series (of 3 patches I think) and make this one a part of the series? Dmitry