Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751705AbbEGSso (ORCPT ); Thu, 7 May 2015 14:48:44 -0400 Received: from mail-gw2-out.broadcom.com ([216.31.210.63]:44565 "EHLO mail-gw2-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751061AbbEGSsm (ORCPT ); Thu, 7 May 2015 14:48:42 -0400 X-IronPort-AV: E=Sophos;i="5.13,384,1427785200"; d="scan'208";a="64214007" Message-ID: <554BB386.4070704@broadcom.com> Date: Thu, 7 May 2015 11:48:38 -0700 From: Ray Jui User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Brian Norris , Arnd Bergmann CC: , Dmitry Torokhov , Anatol Pomazao , Corneliu Doban , Jonathan Richardson , Scott Branden , Florian Fainelli , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , , Dan Ehrenberg , Gregory Fong , , , Kevin Cernekee Subject: Re: [PATCH v3 06/10] mtd: brcmstb_nand: add SoC-specific support References: <1430935194-7579-1-git-send-email-computersforpeace@gmail.com> <7101952.uOJDgn7tgf@wuerfel> <20150506204910.GJ32500@ld-irv-0074> <20781942.cIPodTiNzG@wuerfel> <20150507184246.GO32500@ld-irv-0074> In-Reply-To: <20150507184246.GO32500@ld-irv-0074> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3036 Lines: 74 On 5/7/2015 11:42 AM, Brian Norris wrote: > On Thu, May 07, 2015 at 12:01:02PM +0200, Arnd Bergmann wrote: >> On Wednesday 06 May 2015 13:49:10 Brian Norris wrote: >>> On Wed, May 06, 2015 at 09:12:43PM +0200, Arnd Bergmann wrote: >>>> On Wednesday 06 May 2015 10:59:50 Brian Norris wrote: >>>>> + /* >>>>> + * Some SoCs integrate this controller (e.g., its interrupt bits) in >>>>> + * interesting ways >>>>> + */ >>>>> + if (of_property_read_bool(dn, "brcm,nand-soc")) { >>>>> + struct device_node *soc_dn; >>>>> + >>>>> + soc_dn = of_parse_phandle(dn, "brcm,nand-soc", 0); >>>>> + if (!soc_dn) >>>>> + return -ENODEV; >>>>> + >>>>> + ctrl->soc = devm_brcmnand_probe_soc(dev, soc_dn); >>>>> + if (!ctrl->soc) { >>>>> + dev_err(dev, "could not probe SoC data\n"); >>>>> + of_node_put(soc_dn); >>>>> + return -ENODEV; >>>>> + } >>>>> + >>>>> + ret = devm_request_irq(dev, ctrl->irq, brcmnand_irq, 0, >>>>> + DRV_NAME, ctrl); >>>>> + >>>>> + /* Enable interrupt */ >>>>> + ctrl->soc->ctlrdy_set_enabled(ctrl->soc, true); >>>>> + >>>>> + of_node_put(soc_dn); >>>>> + } else { >>>>> + /* Use standard interrupt infrastructure */ >>>>> + ret = devm_request_irq(dev, ctrl->irq, brcmnand_ctlrdy_irq, 0, >>>>> + DRV_NAME, ctrl); >>>>> + } >>>>> >>>> >>>> It looks to me like this should be handled as a nested irqchip, so the node >>>> you look up gets used as the "interrupt-parent" instead, making the behavior >>>> of this SoC transparent to the nand driver. >>> >>> You snipped the rest of the patch, which involves more than just IRQ >>> handling. The same registers touch both interrupts and data bus endian >>> configuration, so it can't possibly be done transparently to the NAND >>> driver. >> >> Anything else in there? > > Looks like miscellaneous NAND-related control bits. AXI and APB endian > configuration; several interrupt-enable bits (we only use one for now); > a clock-enable; and some timing test mode bits. > >> The bus configuration would just involve writing >> a constant value in some register, right? > > I'm not an expert on the Cygnus/iProc chips, but I believe the answer is > no: we *must* reconfigure the bus before and after each data > transaction, because it affects the rest of the core too. Others on the > CC list can probably elaborate. > Yes, we must configure the bus before the after each data/cache registers access, because it changes the APB bus endianess. Thanks, Ray -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/