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;
Subject: Re: [PATCH 5/6] arm: enable RapidIO menu in Kconfig
To:     Christoph Hellwig <hch@infradead.org>,
        Alexei Colin <acolin@isi.edu>
Cc:     Russell King <linux@armlinux.org.uk>,
        Arnd Bergmann <arnd@arndb.de>,
        Andrew Morton <akpm@linux-foundation.org>,
        John Paul Walters <jwalters@isi.edu>,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
References: <20180730225035.28365-1-acolin@isi.edu>
 <20180730225035.28365-6-acolin@isi.edu> <20180731120428.GA5696@infradead.org>
From:   Alex Bounine <alex.bou9@gmail.com>
Message-ID: <3784698f-bd9d-6d02-6fa4-2f6057e4c93f@gmail.com>
Date:   Tue, 31 Jul 2018 08:43:02 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <20180731120428.GA5696@infradead.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 2018-07-31 08:04 AM, Christoph Hellwig wrote:
> On Mon, Jul 30, 2018 at 06:50:33PM -0400, Alexei Colin wrote:
>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> index afe350e5e3d9..602a61324890 100644
>> --- a/arch/arm/Kconfig
>> +++ b/arch/arm/Kconfig
>> @@ -1278,6 +1278,8 @@ config PCI_HOST_ITE8152
>>   
>>   source "drivers/pci/Kconfig"
>>   
>> +source "drivers/rapidio/Kconfig"
> 
> Please include this from drivers/Kconfig instead of enabling it
> for a completely random set of architectures.
> 
This is not a random set of architectures but only ones that implement 
support for RapidIO as system bus.

On some platforms RapidIO can be the only system bus available replacing 
PCI/PCIe.

As it is done now, RapidIO is configured in "Bus Options" (x86/PPC) or 
"Bus Support" (ARMs) sub-menu and from system configuration option it 
should be kept this way.

Current location of RAPIDIO configuration option is familiar to users of 
PowerPC and x86 platforms, and is similarly available in some ARM 
manufacturers kernel code trees.