From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Franklin S Cooper Jr." Subject: Re: [PATCH 1/2 v2] ARM: DRA7/335x/437x/OMAP4: hwmod: Remove elm address space from hwmod data Date: Mon, 26 Oct 2015 09:50:16 -0500 Message-ID: <562E3DA8.4000006@ti.com> References: <1444926461-16092-1-git-send-email-fcooper@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Paul Walmsley Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, linux@arm.linux.org.uk, tony@atomide.com, rogerq@ti.com, nm@ti.com, lokeshvutla@ti.com List-Id: linux-omap@vger.kernel.org On 10/23/2015 02:00 PM, Paul Walmsley wrote: > On Thu, 15 Oct 2015, Franklin S Cooper Jr wrote: > >> ELM address information is provided by device tree. No longer need >> to include this information within hwmod. >> >> Signed-off-by: Franklin S Cooper Jr > The OMAP4 DTS files don't have the ELM address space declared. I'm going > to drop that portion of your patch. Could you please send a two-patch > series that first adds the ELM address space to the OMAP4 DTS file and > then subsequently drops it from the OMAP4 hwmod data file? Hi Paul, Sorry about that. I can create the patches but I don't see any board omap4 board that has nand support. So I'm not going to be able to test to insure if omap_elm.c will work as is. Should I just drop omap4 from this patchset or should I just add the elm node to omap4.dtsi and if people report an issue with omap_elm then we can fix it? > > > - Paul From mboxrd@z Thu Jan 1 00:00:00 1970 From: fcooper@ti.com (Franklin S Cooper Jr.) Date: Mon, 26 Oct 2015 09:50:16 -0500 Subject: [PATCH 1/2 v2] ARM: DRA7/335x/437x/OMAP4: hwmod: Remove elm address space from hwmod data In-Reply-To: References: <1444926461-16092-1-git-send-email-fcooper@ti.com> Message-ID: <562E3DA8.4000006@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 10/23/2015 02:00 PM, Paul Walmsley wrote: > On Thu, 15 Oct 2015, Franklin S Cooper Jr wrote: > >> ELM address information is provided by device tree. No longer need >> to include this information within hwmod. >> >> Signed-off-by: Franklin S Cooper Jr > The OMAP4 DTS files don't have the ELM address space declared. I'm going > to drop that portion of your patch. Could you please send a two-patch > series that first adds the ELM address space to the OMAP4 DTS file and > then subsequently drops it from the OMAP4 hwmod data file? Hi Paul, Sorry about that. I can create the patches but I don't see any board omap4 board that has nand support. So I'm not going to be able to test to insure if omap_elm.c will work as is. Should I just drop omap4 from this patchset or should I just add the elm node to omap4.dtsi and if people report an issue with omap_elm then we can fix it? > > > - Paul From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754202AbbJZOu6 (ORCPT ); Mon, 26 Oct 2015 10:50:58 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:59844 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753986AbbJZOu4 (ORCPT ); Mon, 26 Oct 2015 10:50:56 -0400 Subject: Re: [PATCH 1/2 v2] ARM: DRA7/335x/437x/OMAP4: hwmod: Remove elm address space from hwmod data To: Paul Walmsley References: <1444926461-16092-1-git-send-email-fcooper@ti.com> CC: , , , , , , , From: "Franklin S Cooper Jr." Message-ID: <562E3DA8.4000006@ti.com> Date: Mon, 26 Oct 2015 09:50:16 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/23/2015 02:00 PM, Paul Walmsley wrote: > On Thu, 15 Oct 2015, Franklin S Cooper Jr wrote: > >> ELM address information is provided by device tree. No longer need >> to include this information within hwmod. >> >> Signed-off-by: Franklin S Cooper Jr > The OMAP4 DTS files don't have the ELM address space declared. I'm going > to drop that portion of your patch. Could you please send a two-patch > series that first adds the ELM address space to the OMAP4 DTS file and > then subsequently drops it from the OMAP4 hwmod data file? Hi Paul, Sorry about that. I can create the patches but I don't see any board omap4 board that has nand support. So I'm not going to be able to test to insure if omap_elm.c will work as is. Should I just drop omap4 from this patchset or should I just add the elm node to omap4.dtsi and if people report an issue with omap_elm then we can fix it? > > > - Paul