From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Bresticker Subject: Re: [PATCH V3 2/4] of: Add binding document for MIPS GIC Date: Wed, 29 Oct 2014 10:08:10 -0700 Message-ID: References: <1414541562-10076-1-git-send-email-abrestic@chromium.org> <1414541562-10076-3-git-send-email-abrestic@chromium.org> <5450CAF9.3040902@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <5450CAF9.3040902@imgtec.com> Sender: linux-kernel-owner@vger.kernel.org To: Qais Yousef Cc: Ralf Baechle , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Thomas Gleixner , Jason Cooper , Daniel Lezcano , John Crispin , David Daney , Linux-MIPS , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: devicetree@vger.kernel.org On Wed, Oct 29, 2014 at 4:09 AM, Qais Yousef wrote: > On 10/29/2014 12:12 AM, Andrew Bresticker wrote: >> >> +- reg : Base address and length of the GIC registers. >> > > Also except for sead3, the base address should be properly reported by the > hardware. The size is fixed (for a specific version of GIC at least - which > is also reported by the hardware). So it would be nice to make this > optional. Even though this is usually probable, I'd prefer to leave this as required, or at least "optional, but recommended". I don't have a very strong opinion on it though, but perhaps the device-tree folks do?