From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Tue, 14 Aug 2018 14:42:05 +0200 (CEST) Received: from mail.bootlin.com ([62.4.15.54]:35245 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S23992747AbeHNMmBcgKyg (ORCPT ); Tue, 14 Aug 2018 14:42:01 +0200 Received: by mail.bootlin.com (Postfix, from userid 110) id 48C4D207CD; Tue, 14 Aug 2018 14:41:54 +0200 (CEST) Received: from localhost (unknown [37.169.111.59]) by mail.bootlin.com (Postfix) with ESMTPSA id D638D2072C; Tue, 14 Aug 2018 14:41:43 +0200 (CEST) Date: Tue, 14 Aug 2018 14:41:35 +0200 From: Alexandre Belloni To: Quentin Schulz Cc: Rob Herring , ralf@linux-mips.org, paul.burton@mips.com, jhogan@kernel.org, mark.rutland@arm.com, davem@davemloft.net, kishon@ti.com, andrew@lunn.ch, f.fainelli@gmail.com, linux-mips@linux-mips.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, allan.nielsen@microsemi.com, thomas.petazzoni@bootlin.com Subject: Re: [PATCH net-next 02/10] dt-bindings: net: ocelot: remove hsio from the list of register address spaces Message-ID: <20180814124135.GK943@piout.net> References: <3558e538b55a2249b0a179c04c27e9d3715bbbaa.1532954208.git-series.quentin.schulz@bootlin.com> <20180813223103.GA16669@rob-hp-laptop> <20180814064953.vboz2gryq4jff34n@qschulz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180814064953.vboz2gryq4jff34n@qschulz> User-Agent: Mutt/1.10.1 (2018-07-13) Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 65588 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: alexandre.belloni@bootlin.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On 14/08/2018 08:49:53+0200, Quentin Schulz wrote: > Understood but it's an intermediate patch. Later (patch 8), the SerDes > muxing "controller" is added as a child to this node. There most likely > will be some others in the future (temperature sensor for example). > > Furthermore, there's already a simple-mfd without children in this file: > https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/mips/mscc.txt#L19 > > How should we handle this case? > There were child nodes in previous version of the binding. You can remove simple-mfd now. The useful registers that are not used by any drivers are gpr and chipid. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com