Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <201803231411.1XijNmJc%fengguang.wu@intel.com>
References: <1521643299-18231-1-git-send-email-roy.pledge@nxp.com> <201803231411.1XijNmJc%fengguang.wu@intel.com>
From:   Li Yang <leoyang.li@nxp.com>
Date:   Fri, 23 Mar 2018 15:04:42 -0500
Message-ID: <CADRPPNTk0_Odjd8CayS4Q1ROHmpYBkxJ7NusCdRB=KfDshO3OA@mail.gmail.com>
Subject: Re: [v2] staging:fsl-mc: Move DPIO from staging to drivers/soc/fsl
To:     Roy Pledge <roy.pledge@nxp.com>
Cc:     kbuild-all@01.org, devel@driverdev.osuosl.org,
        linux-arm-kernel@lists.infradead.org,
        lkml <linux-kernel@vger.kernel.org>, ruxandra.radulescu@nxp.com,
        Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Laurentiu Tudor <laurentiu.tudor@nxp.com>, stuyoder@gmail.com,
        Catalin Marinas <catalin.marinas@arm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Mar 23, 2018 at 1:46 AM, kbuild test robot <lkp@intel.com> wrote:
> Hi Roy,
>
> I love your patch! Yet something to improve:
>
> [auto build test ERROR on staging/staging-testing]
> [also build test ERROR on next-20180322]
> [cannot apply to linus/master v4.16-rc6]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url:    https://github.com/0day-ci/linux/commits/Roy-Pledge/staging-fsl-mc-Move-DPIO-from-staging-to-drivers-soc-fsl/20180323-102325
> config: arm-allmodconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=arm
>
> All errors (new ones prefixed by >>):
>
>    In file included from drivers/crypto/caam/sg_sw_sec4.h:14:0,
>                     from drivers/crypto/caam/caamalg.c:55:
>>> drivers/crypto/caam/sg_sw_qm2.h:38:10: fatal error: ../../../drivers/staging/fsl-mc/include/dpaa2-fd.h: No such file or directory
>     #include "../../../drivers/staging/fsl-mc/include/dpaa2-fd.h"
>              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Hi Roy,

As complained by this auto build error, we should update all the
references to the moved header files.  A quick search shows not only
thecaam driver but also the dpaa2-eth driver need to be updated.

Regards,
Leo

>    compilation terminated.
>
> vim +38 drivers/crypto/caam/sg_sw_qm2.h
>
> 297b9ceb Horia Geantă 2017-07-18  37
> 297b9ceb Horia Geantă 2017-07-18 @38  #include "../../../drivers/staging/fsl-mc/include/dpaa2-fd.h"
> 297b9ceb Horia Geantă 2017-07-18  39
>
> :::::: The code at line 38 was first introduced by commit
> :::::: 297b9cebd2fc020f0bd3e0aac68b0758ab84e8d8 crypto: caam/jr - add support for DPAA2 parts
>
> :::::: TO: Horia Geantă <horia.geanta@nxp.com>
> :::::: CC: Herbert Xu <herbert@gondor.apana.org.au>
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation