mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 0/2] add spi command
Date: Sat, 3 Nov 2012 08:36:56 +0100	[thread overview]
Message-ID: <20121103073656.GW29599@game.jcrosoft.org> (raw)
In-Reply-To: <20121102190141.GN1641@pengutronix.de>

On 20:01 Fri 02 Nov     , Sascha Hauer wrote:
> On Fri, Nov 02, 2012 at 05:06:59PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > HI,
> > 
> > 	while debugging a spi issue I found we have a i2c commands but no spi
> > 	command
> > 
> > 	So introduce one
> > 
> > The following changes since commit 56371f4e8446d04d2a5608dd5890fd87a50f888e:
> > 
> >   iomux-imx35: add missing comma in pad definition (2012-11-01 16:32:12 +0100)
> > 
> > are available in the git repository at:
> > 
> >   git://git.jcrosoft.org/barebox.git delivery/command_spi
> > 
> > for you to fetch changes up to 3d6660a40aa5a51215a1909fd2fb5702f0e6558b:
> > 
> 
> I recently used the i2c_* commands and thought it would be a good idea
> to introduce regmap for barebox. This way we could add a regmap command
> which just maps arbitrary i2c/spi devices as a cdev. Also we could
> replace several drivers in drivers/mfd/ with nearly one-liners.

why not but how to pass all the parameter to this cdev

on SPI we need some on i2c it's more simple

If this will be for the next release ok otherwise can we have this command
until then

Best Regards,
J.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2012-11-03  7:39 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-02 16:06 Jean-Christophe PLAGNIOL-VILLARD
2012-11-02 16:17 ` [PATCH 1/2] spi: introduce spi_get_master Jean-Christophe PLAGNIOL-VILLARD
2012-11-02 16:17   ` [PATCH 2/2] introduce spi command Jean-Christophe PLAGNIOL-VILLARD
2012-11-02 16:26     ` Alexander Shiyan
2012-11-02 16:45       ` Jean-Christophe PLAGNIOL-VILLARD
2012-11-03 21:39     ` Sascha Hauer
2012-11-02 19:01 ` [PATCH 0/2] add " Sascha Hauer
2012-11-03  7:36   ` Jean-Christophe PLAGNIOL-VILLARD [this message]

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=20121103073656.GW29599@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.com \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.de \
    /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