linux-kernel.vger.kernel.org archive mirror
 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: 18+ 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-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-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 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).