Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs

From	Horia Geantă <horia.geanta@nxp.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs
Date	2017-07-20 12:30 +0200
Message-ID	<u5gSt-5Zw-3@gated-at.bofh.it> (permalink)
References	<u4E0N-4S6-7@gated-at.bofh.it> <u4E0P-4S6-53@gated-at.bofh.it> <u4QbE-4dp-15@gated-at.bofh.it> <u4ZHY-2x4-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 7/19/2017 7:04 PM, Logan Gunthorpe wrote:
> 
> 
> On 18/07/17 11:57 PM, Michael Ellerman wrote:
>> Seems fair enough, have you tested it at all?
> 
> It's only been compile tested and the kbuild robot has beat up on it a bit.
> 
Looks like the patch set does not compile on PPC (.config generated
using make corenet64_smp_defconfig):

[...]
  LD      vmlinux.o
  MODPOST vmlinux.o
drivers/crypto/caam/jr.o: In function `rd_reg64':
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:154:
undefined reference to `.ioread64be_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:151:
undefined reference to `.ioread64_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:151:
undefined reference to `.ioread64_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:154:
undefined reference to `.ioread64be_lo_hi'
drivers/crypto/caam/jr.o: In function `wr_reg64':
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:141:
undefined reference to `.iowrite64_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:141:
undefined reference to `.iowrite64_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:144:
undefined reference to `.iowrite64be_lo_hi'
/home/b05471/repos/cryptodev-2.6/drivers/crypto/caam/regs.h:144:
undefined reference to `.iowrite64be_lo_hi'
make: *** [vmlinux] Error 1

Regards,
Horia

Thread

[PATCH v4 0/5] make io{read|write}64 globally usable Logan Gunthorpe <logang@deltatee.com> - 2017-07-18 19:00 +0200
  [PATCH v4 2/5] iomap: introduce io{read|write}64_{lo_hi|hi_lo} Logan Gunthorpe <logang@deltatee.com> - 2017-07-18 19:00 +0200
  [PATCH v4 3/5] io-64-nonatomic: add io{read|write}64[be]{_lo_hi|_hi_lo} macros Logan Gunthorpe <logang@deltatee.com> - 2017-07-18 19:00 +0200
  [PATCH v4 4/5] ntb: ntb_hw_intel: use io-64-nonatomic instead of in-driver hacks Logan Gunthorpe <logang@deltatee.com> - 2017-07-18 19:00 +0200
    RE: [PATCH v4 4/5] ntb: ntb_hw_intel: use io-64-nonatomic instead of in-driver hacks "Allen Hubbe" <Allen.Hubbe@dell.com> - 2017-07-18 20:00 +0200
  [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs Logan Gunthorpe <logang@deltatee.com> - 2017-07-18 19:00 +0200
    Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs Michael Ellerman <mpe@ellerman.id.au> - 2017-07-19 08:00 +0200
      Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can  use readq/writeq defs Logan Gunthorpe <logang@deltatee.com> - 2017-07-19 18:10 +0200
        Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can  use readq/writeq defs Horia Geantă <horia.geanta@nxp.com> - 2017-07-20 12:30 +0200
          Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can  use readq/writeq defs Horia Geantă <horia.geanta@nxp.com> - 2017-07-20 14:40 +0200
            Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can  use readq/writeq defs Logan Gunthorpe <logang@deltatee.com> - 2017-07-20 17:50 +0200
        Re: [PATCH v4 1/5] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs Michael Ellerman <mpe@ellerman.id.au> - 2017-07-20 13:20 +0200

csiph-web