From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1754774AbdBPN2r (ORCPT <rfc822;w@1wt.eu>);
        Thu, 16 Feb 2017 08:28:47 -0500
Received: from mout.kundenserver.de ([217.72.192.74]:60616 "EHLO
        mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1753285AbdBPN2q (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 16 Feb 2017 08:28:46 -0500
From: Arnd Bergmann <arnd@arndb.de>
To: linux-arm-kernel@lists.infradead.org
Cc: Chris Packham <chris.packham@alliedtelesis.co.nz>,
        Andrew Lunn <andrew@lunn.ch>, Jason Cooper <jason@lakedaemon.net>,
        Russell King <linux@armlinux.org.uk>, linux-kernel@vger.kernel.org,
        Gregory Clement <gregory.clement@free-electrons.com>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH v3 5/6] ARM: mvebu: Add driver for mv98dx3236-soc-id
Date: Thu, 16 Feb 2017 14:27:55 +0100
Message-ID: <28604982.PnlvXC13kC@wuerfel>
User-Agent: KMail/5.1.3 (Linux/4.8.0-34-generic; KDE/5.18.0; x86_64; ; )
In-Reply-To: <20170216085041.28337-6-chris.packham@alliedtelesis.co.nz>
References: <20170216085041.28337-1-chris.packham@alliedtelesis.co.nz> <20170216085041.28337-6-chris.packham@alliedtelesis.co.nz>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:erOlkfia1OlI2G4TlAY7tHUiScwobEoi7n6V2T9zK7pZsI2r/z3
 uKrrD8y+SI3I0NSgfi4pdbOj/MTCjZAI65OMX7T9qaBvsXnd4Sy/yOHKCuWLMG069LLO623
 qfeQ64QJybxKsULLJw7bY+muimO4PAXD/qm+oKz+R+Gn3pXipNKOQ2he9MfeMIOm7S4qK4A
 mol3GQFOYH1+WABPeFfpA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:B3XTx+YrpX4=:s8d2LSQIPmaBGtGsLqlkbr
 oPKFRMELs8c2naSuruYv3AejpKEtgzEJNRx672TPxJ45PMeCaricf4gInFnbsD9yMB2Ih8j6d
 UmotU5VzHIZ+5ZPCTOP+4OzNl5BWBL3p5DjxJ5vcseel1q+15yG+TeiuSFul2ZJAABF1njMEO
 8WAW7XqFvX9uHW1w/7wBtHGPlL79qjBbdOeYtoEEoYGtlGgfcgzoy1YKzxFtmOuBDQt+ZUwct
 9Ahg6lUkBM2OddT5aUNQIwOmza9l6YxBNfPEQNGpg/PISODIZB0x2YTbPkwh0jkR6KTMrde+9
 gL5Pj3o4P8VnOh96GHufyJGEVsyqBplmrrc0iTuys2iQXjn33vIs77GCmwpjLcF56JRTXsKQ/
 ciQ+0IxHn4GHpeHI/JkKZ4K88NJKjR9/Cn/fT8AhapL8/JoBJQWMdbHpTRrQ4faKi2obGLJCz
 DikjsUjmM1tT1jsVgCQQTIQ54kMe3qFRIX8jjXfLk4pNIrJJQU1FXs0H/me3oBfKIvrVEnS9D
 VKqMQ+aa5HC1NA5+4J1r3I3QdnzSxG+4e6frEQ9aKvat0g9T9hlL/u4VyZ8XqWFI2KWgh8Lg3
 iJQ/YxiihyoSid1O/bpTEi7b0E+fENxy13qbp1Ce/3CEmqSf2lczJOC1tKq6UL09AsJ8azzOA
 hfPyz5WWDAecjhBHFKgZN80niZlBJpVPk9pI/6VYOkLlZxXCG1Q8iw3+BrWuNi9Yei/GSgtjb
 TPaobrLqK/0rDt9x
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thursday, February 16, 2017 9:50:39 PM CET Chris Packham wrote:
> The DFX server on the 98dx3236 and compatible SoCs has an ID register
> that provides revision information that the PCI based ID register
> doesn't have. Use this if it's available.
> 
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> 

How about putting this new code into a separate driver in
drivers/soc/? I don't think you need the early probing we have
here, and not that much is shared otherwise.

	Arnd