All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Mark Brown <broonie@kernel.org>, James Hogan <jhogan@kernel.org>,
	Paul Burton <paul.burton@mips.com>,
	linux-spi <linux-spi@vger.kernel.org>,
	devicetree <devicetree@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux MIPS Mailing List <linux-mips@linux-mips.org>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Allan Nielsen <allan.nielsen@microsemi.com>,
	Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH 3/5] spi: dw-mmio: add MSCC Ocelot support
Date: Tue, 17 Jul 2018 23:40:24 +0200	[thread overview]
Message-ID: <20180717214024.GD3211@piout.net> (raw)
In-Reply-To: <CAHp75Vcvqqv4kHHDE2wbtVFFa3a5FLqg8meDeOa59-qaDFGFTw@mail.gmail.com>

On 18/07/2018 00:34:37+0300, Andy Shevchenko wrote:
> On Tue, Jul 17, 2018 at 5:23 PM, Alexandre Belloni
> <alexandre.belloni@bootlin.com> wrote:
> > Because the SPI controller deasserts the chip select when the TX fifo is
> > empty (which may happen in the middle of a transfer), the CS should be
> > handled by linux. Unfortunately, some or all of the first four chip
> > selects are not muxable as GPIOs, depending on the SoC.
> >
> > There is a way to bitbang those pins by using the SPI boot controller so
> > use it to set the chip selects.
> >
> > At init time, it is also necessary to give control of the SPI interface to
> > the Designware IP.
> 
> > +       ret = dw_spi_mscc_init(pdev, dwsmmio);
> > +       if (ret)
> > +               goto out;
> 
> > +       { .compatible = "mscc,ocelot-spi", .data = dw_spi_mscc_init},
> 
> Looks like you were thinking about something like
> 
> init_func = device_get_match_data(...);
> if (init_func) {
>  ret = init_func();
>  if (ret)
>    return ret;
> }
> 
> ?
> 

Ah sure, I forgot to do that after testing.

-- 
Alexandre Belloni, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2018-07-17 21:40 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-17 14:23 [PATCH 0/5] Add support for MSCC Ocelot SPI Alexandre Belloni
2018-07-17 14:23 ` [PATCH 1/5] spi: dw: fix possible race condition Alexandre Belloni
2018-07-17 21:30   ` Andy Shevchenko
2018-07-17 21:30     ` Andy Shevchenko
2018-07-17 21:42       ` Alexandre Belloni
2018-07-17 21:54         ` Andy Shevchenko
2018-07-17 22:13           ` Alexandre Belloni
2018-07-18 10:33             ` Andy Shevchenko
2018-07-18 10:51   ` Mark Brown
2018-07-18 12:29   ` Applied "spi: dw: fix possible race condition" to the spi tree Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-17 14:23 ` [PATCH 2/5] spi: dw: allow providing own set_cs callback Alexandre Belloni
2018-07-18 12:24   ` Mark Brown
2018-07-18 12:29   ` Applied "spi: dw: allow providing own set_cs callback" to the spi tree Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-18 12:29     ` Mark Brown
2018-07-17 14:23 ` [PATCH 3/5] spi: dw-mmio: add MSCC Ocelot support Alexandre Belloni
2018-07-17 21:34   ` Andy Shevchenko
2018-07-17 21:40     ` Alexandre Belloni [this message]
2018-07-17 14:23 ` [PATCH 4/5] mips: dts: mscc: Add spi on Ocelot Alexandre Belloni
2018-07-17 14:23 ` [PATCH 5/5] mips: dts: mscc: enable spi and NOR flash support on ocelot PCB123 Alexandre Belloni

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=20180717214024.GD3211@piout.net \
    --to=alexandre.belloni@bootlin.com \
    --cc=allan.nielsen@microsemi.com \
    --cc=andy.shevchenko@gmail.com \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jhogan@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@linux-mips.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=paul.burton@mips.com \
    --cc=robh+dt@kernel.org \
    --cc=thomas.petazzoni@bootlin.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.