From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756508AbcBHViE (ORCPT ); Mon, 8 Feb 2016 16:38:04 -0500 Received: from mail.kernel.org ([198.145.29.136]:39097 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756210AbcBHViC (ORCPT ); Mon, 8 Feb 2016 16:38:02 -0500 Date: Mon, 8 Feb 2016 15:37:57 -0600 From: Rob Herring To: Masahiro Yamada Cc: Arnd Bergmann , linux-mtd@lists.infradead.org, Dinh Nguyen , devicetree@vger.kernel.org, Kumar Gala , Linux Kernel Mailing List , Ian Campbell , Pawel Moll , Mark Rutland Subject: Re: [PATCH] mtd: denali: fix property name for Denali DT binding Message-ID: <20160208213757.GA3223@rob-hp-laptop> References: <1454916702-20982-1-git-send-email-yamada.masahiro@socionext.com> <5298318.0qJ9fLYbSx@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 08, 2016 at 05:41:33PM +0900, Masahiro Yamada wrote: > Hi Arnd, > > > 2016-02-08 17:29 GMT+09:00 Arnd Bergmann : > > On Monday 08 February 2016 16:31:42 Masahiro Yamada wrote: > >> diff --git a/Documentation/devicetree/bindings/mtd/denali-nand.txt b/Documentation/devicetree/bindings/mtd/denali-nand.txt > >> index b04d03a..785b825 100644 > >> --- a/Documentation/devicetree/bindings/mtd/denali-nand.txt > >> +++ b/Documentation/devicetree/bindings/mtd/denali-nand.txt > >> @@ -5,7 +5,7 @@ Required properties: > >> - reg : should contain registers location and length for data and reg. > >> - reg-names: Should contain the reg names "nand_data" and "denali_reg" > >> - interrupts : The interrupt number. > >> - - dm-mask : DMA bit mask > >> + - dma-mask : DMA bit mask > >> > >> The device tree may optionally contain sub-nodes describing partitions of the > >> address space. See partition.txt for more detail. > >> > > > > It looks like this binding is wrong in multiple ways, and it doesn't seem to > > be used in any .dts files. Is this actually being shipped anywhere or > > could we try to fix the binding properly? > > > > Looks like it is locally used in Altera's Rocketboard tree now. > > See this: > > https://github.com/altera-opensource/linux-socfpga/blob/socfpga-4.3/arch/arm/boot/dts/socfpga.dtsi > > > I hope Dinh can comment on the status. >>From the looks of it, this property should be dropped and the driver should be updated to not touch dev->dma_mask. The core sets it up correctly now. Rob