linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Pantelis Antoniou <panto@antoniou-consulting.com>
Cc: Grant Likely <grant.likely@secretlab.ca>,
	Rob Herring <rob.herring@calxeda.com>,
	Rob Landley <rob@landley.net>, Jon Loeliger <jdl@jdl.com>,
	Stephen Warren <swarren@wwwdotorg.org>,
	David Gibson <david@gibson.dropbear.id.au>,
	Benoit Cousson <b-cousson@ti.com>,
	Mitch Bradley <wmb@firmworks.com>, Alan Tull <atull@altera.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-omap@vger.kernel.org, devicetree-discuss@lists.ozlabs.org,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Matt Porter <mporter@ti.com>, Russ Dill <Russ.Dill@ti.com>,
	Koen Kooi <koen@dominion.thruhere.net>,
	Joel A Fernandes <agnel.joel@gmail.com>,
	Rob Clark <robdclark@gmail.com>,
	Jason Kridner <jkridner@beagleboard.org>,
	Matt Ranostay <mranostay@gmail.com>
Subject: Re: [PATCH 1/5] capemgr: Beaglebone DT overlay based cape manager
Date: Mon, 7 Jan 2013 13:05:52 -0800	[thread overview]
Message-ID: <20130107210551.GL14149@atomide.com> (raw)
In-Reply-To: <93BF5C62-ADE8-4EFC-9175-C2D7E43300E3@antoniou-consulting.com>

* Pantelis Antoniou <panto@antoniou-consulting.com> [130107 12:43]:
> Hi Tony,
> 
> On Jan 7, 2013, at 10:35 PM, Tony Lindgren wrote:
> 
> > * Pantelis Antoniou <panto@antoniou-consulting.com> [130107 12:29]:
> >> On Jan 7, 2013, at 10:23 PM, Tony Lindgren wrote:
> >>> 
> >>> Well how about split it to an eeprom driver, and Linux generic
> >>> device loader parts?
> >>> 
> >> 
> >> All that's left is the eeprom driver (accessor) and calls to the 
> >> generic DT overlay constructs. 
> >> 
> >> If you caught on the previous patchset about DT overlays it should be
> >> clear.
> >> 
> >> So it is split along those lines already.
> > 
> > Hmm I was thinking something like this:
> > 
> > drivers/base/device-loader.c
> > drivers/misc/eeprom/beaglebone-cape.c
> > 
> > Then you may be able to just load the configuration for it
> > from a .dts file and maybe no hardware specific glue is even
> > needed.
> > 
> 
> At the end of the line, some kind of hardware glue is going to be needed.
> 
> I just feel that drawing from a sample size of 1 (maybe 2 if I get to throw
> in the beagleboard), it is a bit premature to think about making it overly
> general, besides the part that are obviously part of the infrastructure 
> (like the DT overlay stuff).
> 
> What I'm getting at, is that we need some user experience about this, before
> going away and creating structure out of possible misconception about the uses. 

IMHO stuff like this will be needed by many SoCs. Some examples of similar
things for omaps that have eventually become generic frameworks have been
the clock framework, USB OTG support, runtime PM, pinmux framework and
so on.

So I suggest a minimal generic API from the start as that will make things
a lot easier in the long run.

Regards,

Tony

  reply	other threads:[~2013-01-07 21:05 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-07 18:51 [PATCH 0/5] DT Overlay based cape manager for TI's Beaglebone Pantelis Antoniou
2013-01-07 18:51 ` [PATCH 1/5] capemgr: Beaglebone DT overlay based cape manager Pantelis Antoniou
2013-01-07 20:09   ` Tony Lindgren
2013-01-07 20:13     ` Pantelis Antoniou
2013-01-07 20:23       ` Tony Lindgren
     [not found]         ` <20130107202306.GH14149-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2013-01-07 20:26           ` Pantelis Antoniou
2013-01-07 20:35             ` Tony Lindgren
2013-01-07 20:40               ` Pantelis Antoniou
2013-01-07 21:05                 ` Tony Lindgren [this message]
2013-01-07 21:35                   ` Arnd Bergmann
2013-01-08  9:15                     ` Pantelis Antoniou
2013-01-08  9:51                       ` Guennadi Liakhovetski
2013-01-08 10:07                         ` Pantelis Antoniou
2013-01-08 10:00                       ` Lee Jones
2013-01-08 10:10                         ` Pantelis Antoniou
2013-01-08 10:48                           ` Lee Jones
2013-01-08 12:12                             ` Arnd Bergmann
2013-01-08 13:26                               ` Pantelis Antoniou
2013-01-08 16:12                                 ` Arnd Bergmann
2013-01-09  8:11                                   ` Lee Jones
2013-01-09  8:29                                     ` Linus Walleij
     [not found]                                       ` <CACRpkdbx7ptCugpc1+JC5Yk+n835goOuoF6q0pdizsjpZ-G9mQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-01-09  9:56                                         ` Pantelis Antoniou
     [not found]                                           ` <57EDCDB4-2AC9-4052-BBA6-9F4A5D3C3D8C-wVdstyuyKrO8r51toPun2/C9HSW9iNxf@public.gmane.org>
2013-01-09 10:21                                             ` Arnd Bergmann
     [not found]                                               ` <201301091021.24147.arnd-r2nGTMty4D4@public.gmane.org>
2013-01-09 10:24                                                 ` Pantelis Antoniou
2013-01-09 11:48                                         ` Arnd Bergmann
     [not found]                                           ` <201301091148.09320.arnd-r2nGTMty4D4@public.gmane.org>
2013-01-17 10:20                                             ` Linus Walleij
     [not found]                                               ` <CACRpkdZ8hgV_=ev8Kcq=i7K15jrvwW+Or7p+=8fP+8Rb7GGvTQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-01-17 10:35                                                 ` Arnd Bergmann
     [not found]                                                   ` <201301171035.41407.arnd-r2nGTMty4D4@public.gmane.org>
2013-01-17 14:02                                                     ` Linus Walleij
2013-03-26 16:16                           ` Grant Likely
2013-03-26 18:40                             ` Greg Kroah-Hartman
2013-01-08 11:14                     ` Sascha Hauer
     [not found]                     ` <201301072135.05166.arnd-r2nGTMty4D4@public.gmane.org>
2013-04-15 10:14                       ` Jean-Christophe PLAGNIOL-VILLARD
2013-01-07 18:51 ` [PATCH 2/5] capemgr: Add beaglebone's cape driver bindings Pantelis Antoniou
2013-03-26 17:36   ` Grant Likely
2013-03-27  9:26     ` Pantelis Antoniou
     [not found] ` <1357584666-17374-1-git-send-email-panto-wVdstyuyKrO8r51toPun2/C9HSW9iNxf@public.gmane.org>
2013-01-07 18:51   ` [PATCH 3/5] capemgr: am335x-bone capemgr bindings Pantelis Antoniou
2013-01-07 18:51 ` [PATCH 4/5] capemgr: firmware makefiles for DT objects Pantelis Antoniou
2013-01-07 18:51 ` [PATCH 5/5] capemgr: Weather cape cape definition Pantelis Antoniou

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130107210551.GL14149@atomide.com \
    --to=tony@atomide.com \
    --cc=Russ.Dill@ti.com \
    --cc=agnel.joel@gmail.com \
    --cc=arnd@arndb.de \
    --cc=atull@altera.com \
    --cc=b-cousson@ti.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=gregkh@linuxfoundation.org \
    --cc=jdl@jdl.com \
    --cc=jkridner@beagleboard.org \
    --cc=koen@dominion.thruhere.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mporter@ti.com \
    --cc=mranostay@gmail.com \
    --cc=panto@antoniou-consulting.com \
    --cc=rob.herring@calxeda.com \
    --cc=rob@landley.net \
    --cc=robdclark@gmail.com \
    --cc=swarren@wwwdotorg.org \
    --cc=wmb@firmworks.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).