Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751676AbdGROjF (ORCPT <rfc822;w@1wt.eu>);
        Tue, 18 Jul 2017 10:39:05 -0400
Received: from mail-oi0-f53.google.com ([209.85.218.53]:35967 "EHLO
        mail-oi0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751455AbdGROjE (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 18 Jul 2017 10:39:04 -0400
MIME-Version: 1.0
In-Reply-To: <596E1CD7.1050909@nxp.com>
References: <20170718133723.12709-1-laurentiu.tudor@nxp.com>
 <20170718133723.12709-9-laurentiu.tudor@nxp.com> <CAK8P3a0WvNgk5eXQoPpXiLwOS_wB1NdfCyPtiKMys3qqEwHjDw@mail.gmail.com>
 <596E1CD7.1050909@nxp.com>
From: Arnd Bergmann <arnd@arndb.de>
Date: Tue, 18 Jul 2017 16:39:03 +0200
X-Google-Sender-Auth: CuVfVjYIfNykpFu6idkS0fz5KzQ
Message-ID: <CAK8P3a2E64O6tgAdGck9ce1a=1yeQfzc9tNex7H+i1K0uKaOHg@mail.gmail.com>
Subject: Re: [PATCH v2 8/8] staging: fsl-mc: allow the driver compile multi-arch
To: Laurentiu Tudor <laurentiu.tudor@nxp.com>
Cc: gregkh <gregkh@linuxfoundation.org>, Stuart Yoder <stuyoder@gmail.com>,
        "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Marc Zyngier <marc.zyngier@arm.com>, Alexander Graf <agraf@suse.de>,
        Robin Murphy <robin.murphy@arm.com>,
        Ioana Ciornei <ioana.ciornei@nxp.com>,
        Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>,
        Bharat Bhushan <bharat.bhushan@nxp.com>,
        Catalin Horghidan <catalin.horghidan@nxp.com>,
        Leo Li <leoyang.li@nxp.com>, Roy Pledge <roy.pledge@nxp.com>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 613
Lines: 20

On Tue, Jul 18, 2017 at 4:36 PM, Laurentiu Tudor
<laurentiu.tudor@nxp.com> wrote:

> Good point, I'll take care of it. But don't you mean COMPILE_TEST be
> added on the actual MC_BUS config, like so:
>
>   config FSL_MC_BUS
>         bool "QorIQ DPAA2 fsl-mc bus driver"
> -       depends on OF && ARCH_LAYERSCAPE
> +       depends on OF && (ARCH_LAYERSCAPE || COMPILE_TEST)
>         select GENERIC_MSI_IRQ_DOMAIN
> ?
>
> The other drivers that depend on the MC_BUS won't compile on other
> architectures.

I was thinking of adding it to all three, but you are right we only needed
it for the bus.

        Arnd