Messages in this thread | | | Date | Mon, 20 Mar 2017 13:05:51 -0700 | From | Brian Norris <> | Subject | Re: [4/4] mwifiex: pcie: de-duplicate buffer allocation code |
| |
Hi Kalle,
On Mon, Mar 20, 2017 at 05:08:35PM +0000, Kalle Valo wrote: > Brian Norris <briannorris@chromium.org> wrote: > > This code was duplicated as part of the PCIe FLR code added to this > > driver. Let's de-duplicate it to: > > > > * make things easier to read (mwifiex_pcie_free_buffers() now has a > > corresponding mwifiex_pcie_alloc_buffers()) > > * reduce likelihood of bugs > > * make error logging equally verbose > > * save lines of code! > > > > Also drop some of the commentary that isn't really needed. > > > > Signed-off-by: Brian Norris <briannorris@chromium.org> > > Failed to apply: > > fatal: sha1 information is lacking or useless (drivers/net/wireless/marvell/mwifiex/pcie.c). > error: could not build fake ancestor > Applying: mwifiex: pcie: de-duplicate buffer allocation code > Patch failed at 0001 mwifiex: pcie: de-duplicate buffer allocation code > The copy of the patch that failed is found in: .git/rebase-apply/patch > > Patch set to Changes Requested.
This applies fine to your wireless-drivers/master branch for me, where patches 1-3 were applied. Are you applying this to wireless-drivers-next? It's quite understandable that patch 4 wouldn't apply there, as you've stripped out the previous patches...
Brian
| |