Message-id: <4FA3F6C2.9010602@denx.de>
Date:	Fri, 04 May 2012 17:33:22 +0200
From:	Heiko Schocher <hs@...x.de>
To:	davinci-linux-open-source@...ux.davincidsp.com
Cc:	linux-arm-kernel@...ts.infradead.org,
	devicetree-discuss@...ts.ozlabs.org, linux-mtd@...ts.infradead.org,
	linux-i2c@...r.kernel.org, netdev@...r.kernel.org,
	David Woodhouse <dwmw2@...radead.org>,
	Ben Dooks <ben-linux@...ff.org>,
	Wolfram Sang <w.sang@...gutronix.de>,
	Sekhar Nori <nsekhar@...com>, Kevin Hilman <khilman@...com>,
	Wolfgang Denk <wd@...x.de>,
	Sergei Shtylyov <sshtylyov@...sta.com>
Subject: Re: [PATCH v3 0/7] ARM: davinci: add support for the am1808 based
 enbw_cmc board

Hello,

this v3 patchset is now pending for more than 1 month without
seeing comments for it. Are there no more issues?

Should I rebase it (if no further comments occur), as it is
pending so long? (If so, against which tree?)

Thanks.

bye,
Heiko

Heiko Schocher wrote:
> This patchserie add support for the davinci am1808 based
> enbw_cmc board.
> 
> Important: I rebased this patchserie against the irqdomain/next
> branch from grant likely, as he suggested to rework the OF
> intcontroller changes to the irqdomain work, branch found here:
> 
> http://git.secretlab.ca/?p=linux-2.6.git;a=shortlog;h=refs/heads/irqdomain/next
> 
>     git://git.secretlab.ca/git/linux-2.6.git irqdomain/next
> 
>     commit 280ad7fda5f95211857fda38960f2b6fdf6edd3e
>     Author: Grant Likely <grant.likely@...retlab.ca>
>     Date:   Fri Feb 24 14:58:54 2012 -0700
> 
>     mfd: twl-core: Add IRQ_DOMAIN dependency
> 
> changes for v2:
> Post this patchserie now as v2, as reworked in the
> comments I got for the RFC serie.
> 
> changes for v3:
> - Interrupt Controller:
>   - comment from Sergei Shtylyov:
>     - rename compatible" prop to "ti,cp_intc"
>     - cp_intc_init() is now also for the of case
>       the name of the init function (it calls the
>       "new" __cp_intc_init() function, which was
>       the "old" cp_intc_init()). Through this
>       rework the changes for OF is better visible.
>       As the OF case uses the irq_domain rework from
>       Grant Likely, maybe the none OF case can use
>       this also, but this should be tested on a hw ...
>  
> Got no comments to the following points, I noted in the
> RFC series, so posting this patchseries with them:
> 
> - ARM: davinci: configure davinci aemif chipselects through OF
>   not moved to mfd, as mentioned in this discussion:
>   http://davinci-linux-open-source.1494791.n2.nabble.com/PATCH-arm-davinci-configure-davinci-aemif-chipselects-through-OF-td7059739.html
>   instead use a phandle in the DTS, so drivers which
>   uses the davinci aemif, can call davinci_aemif_setup_timing_of()
> 
>   This is just thought as an RFC ... The enbw_cmc board
>   support not really need to setup this bus timings, as
>   they are setup in U-Boot ... but I want to post this,
>   as I think, it is a nice to have, and I am not really
>   sure, if this has to be a MFD device (If so, all bus
>   interfaces for other SoCs should be converted also to
>   MFD devices) ... as an example how this can be used
>   I add this to the davinci nand controller OF support
>   patch, in this patchserie.
> 
> - ARM: davinci: mux: add OF support
>   I want to get rid of the pin setup code in board code ...
>   This patch introduces a davinci_cfg_reg_of() function,
>   which davinci drivers can call, if they found a
>   "pinmux-handle", so used in the following drivers in
>   this patchserie:
> 
>   drivers/net/ethernet/ti/davinci_emac
>   drivers/i2c/busses/i2c-davinci.c
>   drivers/mtd/nand/davinci_nand.c
> 
> - post this board support with USB support, even though
>   USB is only working with the 10 ms "workaround", posted here:
>   http://comments.gmane.org/gmane.linux.usb.general/54505
>   I see this issue also on the AM1808 TMDXEXP1808L evalboard.
> 
> - MMC and USB are not using OF support yet, ideas how to port
>   this are welcome. I need for USB and MMC board specific
>   callbacks, how to solve this with OF support?
> 
> Signed-off-by: Heiko Schocher <hs@...x.de>
> Cc: linux-arm-kernel@...ts.infradead.org
> Cc: devicetree-discuss@...ts.ozlabs.org
> Cc: davinci-linux-open-source@...ux.davincidsp.com
> Cc: linux-mtd@...ts.infradead.org
> Cc: linux-i2c@...r.kernel.org
> Cc: netdev@...r.kernel.org
> Cc: David Woodhouse <dwmw2@...radead.org>
> Cc: Ben Dooks <ben-linux@...ff.org>
> Cc: Wolfram Sang <w.sang@...gutronix.de>
> Cc: Sekhar Nori <nsekhar@...com>
> Cc: Kevin Hilman <khilman@...com>
> Cc: Wolfgang Denk <wd@...x.de>
> Cc: Sergei Shtylyov <sshtylyov@...sta.com>
> 
> Heiko Schocher (7):
>   ARM: davinci, intc: Add OF support for TI interrupt controller
>   ARM: davinci: configure davinci aemif chipselects through OF
>   ARM: davinci: mux: add OF support
>   ARM: davinci: net: davinci_emac: add OF support
>   ARM: davinci: i2c: add OF support
>   ARM: mtd: nand: davinci: add OF support for davinci nand controller
>   ARM: davinci: add support for the am1808 based enbw_cmc board
> 
>  .../devicetree/bindings/arm/davinci/aemif.txt      |  119 ++++++
>  .../bindings/arm/davinci/davinci_emac.txt          |   43 +++
>  .../devicetree/bindings/arm/davinci/i2c.txt        |   33 ++
>  .../devicetree/bindings/arm/davinci/intc.txt       |   27 ++
>  .../devicetree/bindings/arm/davinci/mux.txt        |   40 ++
>  .../devicetree/bindings/arm/davinci/nand.txt       |   74 ++++
>  arch/arm/boot/dts/enbw_cmc.dts                     |  268 ++++++++++++++
>  arch/arm/configs/enbw_cmc_defconfig                |  123 +++++++
>  arch/arm/mach-davinci/Kconfig                      |    9 +
>  arch/arm/mach-davinci/Makefile                     |    1 +
>  arch/arm/mach-davinci/aemif.c                      |   86 +++++-
>  arch/arm/mach-davinci/board-enbw-cmc.c             |  380 ++++++++++++++++++++
>  arch/arm/mach-davinci/cp_intc.c                    |   87 ++++-
>  arch/arm/mach-davinci/include/mach/aemif.h         |    1 +
>  arch/arm/mach-davinci/include/mach/mux.h           |    2 +
>  arch/arm/mach-davinci/include/mach/uncompress.h    |    1 +
>  arch/arm/mach-davinci/mux.c                        |   73 ++++-
>  drivers/i2c/busses/i2c-davinci.c                   |   37 ++
>  drivers/mtd/nand/davinci_nand.c                    |   85 +++++-
>  drivers/net/ethernet/ti/davinci_emac.c             |   94 +++++-
>  20 files changed, 1569 insertions(+), 14 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/aemif.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/davinci_emac.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/i2c.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/intc.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/mux.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/davinci/nand.txt
>  create mode 100644 arch/arm/boot/dts/enbw_cmc.dts
>  create mode 100644 arch/arm/configs/enbw_cmc_defconfig
>  create mode 100644 arch/arm/mach-davinci/board-enbw-cmc.c
> 

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@...x.de
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists