From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756315AbaDPSmw (ORCPT ); Wed, 16 Apr 2014 14:42:52 -0400 Received: from lxorguk.ukuu.org.uk ([81.2.110.251]:46965 "EHLO lxorguk.ukuu.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755642AbaDPSmr (ORCPT ); Wed, 16 Apr 2014 14:42:47 -0400 Date: Wed, 16 Apr 2014 19:41:09 +0100 From: One Thousand Gnomes To: Sergei Ianovich Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Randy Dunlap , Russell King , Arnd Bergmann , Greg Kroah-Hartman , Grant Likely , Heikki Krogerus , devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND...), linux-doc@vger.kernel.org (open list:DOCUMENTATION) Subject: Re: [PATCH v4 13/21] misc: support for LP-8x4x custom parallel bus Message-ID: <20140416194109.3ef914d2@alan.etchedpixels.co.uk> In-Reply-To: <1397668667-27328-7-git-send-email-ynvich@gmail.com> References: <1397668411-27162-7-git-send-email-ynvich@gmail.com> <1397668667-27328-1-git-send-email-ynvich@gmail.com> <1397668667-27328-7-git-send-email-ynvich@gmail.com> Organization: Intel Corporation X-Mailer: Claws Mail 3.8.1 (GTK+ 2.24.20; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 16 Apr 2014 21:17:18 +0400 Sergei Ianovich wrote: > This patch implements probing for the bus and reporting the number > of available expansion slots. This appears to be a bus not a misc device. I don't think it belongs in misc. As you've got devices on this bus (or nailed into the 'bus' driver) perhaps this belongs as drivers/platform/lp8x4x or similar with any specific drivers split out - or even drivers/lp8x4x if there will be more of them and more than one platform using it ? Alan From mboxrd@z Thu Jan 1 00:00:00 1970 From: One Thousand Gnomes Subject: Re: [PATCH v4 13/21] misc: support for LP-8x4x custom parallel bus Date: Wed, 16 Apr 2014 19:41:09 +0100 Message-ID: <20140416194109.3ef914d2@alan.etchedpixels.co.uk> References: <1397668411-27162-7-git-send-email-ynvich@gmail.com> <1397668667-27328-1-git-send-email-ynvich@gmail.com> <1397668667-27328-7-git-send-email-ynvich@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1397668667-27328-7-git-send-email-ynvich@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Sergei Ianovich Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Randy Dunlap , Russell King , Arnd Bergmann , Greg Kroah-Hartman , Grant Likely , Heikki Krogerus , "open list:OPEN FIRMWARE AND..." , "open list:DOCUMENTATION" List-Id: devicetree@vger.kernel.org On Wed, 16 Apr 2014 21:17:18 +0400 Sergei Ianovich wrote: > This patch implements probing for the bus and reporting the number > of available expansion slots. This appears to be a bus not a misc device. I don't think it belongs in misc. As you've got devices on this bus (or nailed into the 'bus' driver) perhaps this belongs as drivers/platform/lp8x4x or similar with any specific drivers split out - or even drivers/lp8x4x if there will be more of them and more than one platform using it ? Alan From mboxrd@z Thu Jan 1 00:00:00 1970 From: gnomes@lxorguk.ukuu.org.uk (One Thousand Gnomes) Date: Wed, 16 Apr 2014 19:41:09 +0100 Subject: [PATCH v4 13/21] misc: support for LP-8x4x custom parallel bus In-Reply-To: <1397668667-27328-7-git-send-email-ynvich@gmail.com> References: <1397668411-27162-7-git-send-email-ynvich@gmail.com> <1397668667-27328-1-git-send-email-ynvich@gmail.com> <1397668667-27328-7-git-send-email-ynvich@gmail.com> Message-ID: <20140416194109.3ef914d2@alan.etchedpixels.co.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 16 Apr 2014 21:17:18 +0400 Sergei Ianovich wrote: > This patch implements probing for the bus and reporting the number > of available expansion slots. This appears to be a bus not a misc device. I don't think it belongs in misc. As you've got devices on this bus (or nailed into the 'bus' driver) perhaps this belongs as drivers/platform/lp8x4x or similar with any specific drivers split out - or even drivers/lp8x4x if there will be more of them and more than one platform using it ? Alan