Re: [PATCH RFC 1/2] drivers: pci: fix window allocation order wrt bus_range filtering

From: Liviu Dudau
Date: Fri Oct 24 2014 - 05:04:25 EST


On Thu, Oct 23, 2014 at 11:27:07PM +0100, Bjorn Helgaas wrote:
> On Thu, Oct 23, 2014 at 04:23:06PM +0100, Lorenzo Pieralisi wrote:
> > The number of windows allocated for the host bridge depends on the
> > bus resource. Instead of first allocating the windows and then
> > limit the bus resource, this patch reshuffles the code so that if any
> > limitation is applied to the bus resource it is taken into account in
> > the windows allocation.
> >
> > Cc: Will Deacon <will.deacon@xxxxxxx>
> > Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
>
> Hi Lorenzo,
>
> I can *read* your patches just fine, but when I save them using mutt to
> apply them, they look like this:
>
> --- a/drivers/pci/host/pci-host-generic.c
> +++ b/drivers/pci/host/pci-host-generic.c
> @@ -276,17 +276,17 @@ static int gen_pci_parse_map_cfg_windows(struct gen_p=
> ci *pci)
> =09=09return err;
> =09}
> =20
> -=09pci->cfg.win =3D devm_kcalloc(dev,
> resource_size(&pci->cfg.bus_range),
>
> I can work around this by downloading them another way, but it is a
> bit of a hassle. I *think* this is because your email has
>
> Content-Type: text/plain; charset=WINDOWS-1252
>
> which apparently confuses mutt. Any idea why this is? Anybody know how I
> can fix mutt to deal with this?

I use decode-save (<Esc>s on my setup) to save the attachment(s). Sometimes
it works well enough, although more often than not tabs get lost.

Best regards,
Liviu

>
> Bjorn
>

--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
Â\_(ã)_/Â

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/