On Fri, Jun 10, 2016 at 08:56:59PM +0200, Alexander Gordeev wrote:
> On Mon, Jun 06, 2016 at 04:12:03PM +0200, Andrew Jones wrote:
> > > diff --git a/lib/x86/asm/pci.h b/lib/x86/asm/pci.h
> > > index 821a2c1e180a..813dc28b9232 100644
> > > --- a/lib/x86/asm/pci.h
> > > +++ b/lib/x86/asm/pci.h
> > > @@ -35,4 +35,10 @@ static inline void pci_config_writel(pcidevaddr_t dev, uint8_t reg, uint32_t val
> > > outl(val, 0xCFC);
> > > }
> > >
> > > +static inline
> > > +phys_addr_t pci_translate_addr(pcidevaddr_t __unused dev, uint64_t addr)
> >
> > I put the __unused after the argument name.
> >
> > > +{
> > > + return addr;
> >
> > Need tab here.
>
> It will differ then from the coding style of the other functions
> in this header (and elsewhere in x86).
ah, nevermind then
>
> > > +}
> > > +
> > > #endif
> > > --
> > > 1.8.3.1
> > >
> >
> > Thanks,
> > drew
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
