linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: boris.brezillon@free-electrons.com (Boris Brezillon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/11] ARM: at91: remove !DT support for at91rm9200
Date: Fri, 28 Nov 2014 00:39:38 +0100	[thread overview]
Message-ID: <20141128003938.7efb4ea0@bbrezillon> (raw)
In-Reply-To: <3935244.Dm2HkZj2Bg@wuerfel>

Hi Arnd,

On Fri, 28 Nov 2014 00:12:25 +0100
Arnd Bergmann <arnd@arndb.de> wrote:

> On Thursday 27 November 2014 18:12:43 Alexandre Belloni wrote:
> > 
> > As discussed some weeks ago, I prepared patches to switch sama5d[3-4] to
> > multiplatform. We are still missing the SMC and matrix drivers to switch
> > sam9 and rm9200.
> 
> I just looked at the drivers because I got curious, and to see if
> there are still any low-hanging fruit, but I guess you already picked
> them all ;-)
> 
> > The currently affected drivers are:
> >  - drivers/ata/pata_at91.c (SMC)
> >  - drivers/pcmcia/at91_cf.c (SMC)
> 
> I guess the SMC should live in drivers/memory with an interface
> similar to mvebu-devbus.c?

Actually, there's some work in progress to support the EBI/SMC blocks
as a memory controller driver ;-):

http://thread.gmane.org/gmane.linux.kernel/1822096

I'll post a new version soon.

> 
> Seems doable but nontrivial.
> 
> >  - drivers/usb/gadget/udc/at91_udc.c (Matrix, this is the only one
> >    for sam9)
> 
> Is at91_matrix a pin controller? With the board files removed, the udc
> driver has the only two remaining calls to at91_matrix_{read,write}
> for setting the pullup, so that could be modeled as a trivial pinctrl
> driver

The matrix block is containing several system configuration registers.
Most of them are related to AHB/APB bus config (master <-> slave
priority, burst and some other configs I don't remember).
Another register is here to define which HW logic is attached to an
external device connected through the EBI (External Bus Interface):
NAND, SDRAM, CompactFlash, ...
And, as you pointed out, there's a register to configure the pullup of
the UDC device.

As you can see, the matrix registers might be accessed by different
drivers (include the EBI/SMC driver), hence I proposed to expose them as
a syscon device (see the EBI/SMC series).

Best Regards,

Boris

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2014-11-27 23:39 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-27 16:06 [PATCH 00/11] ARM: at91: remove !DT support for at91rm9200 Nicolas Ferre
2014-11-27 16:06 ` [PATCH 01/11] ARM: at91: remove at91rm9200 legacy boards files Nicolas Ferre
2014-11-27 16:06 ` [PATCH 02/11] ARM: at91: remove at91rm9200 legacy board support Nicolas Ferre
2014-11-27 16:06 ` [PATCH 03/11] ARM: at91: switch configuration option to SOC_AT91RM9200 Nicolas Ferre
2014-11-27 16:06 ` [PATCH 04/11] ARM: at91/Kconfig: remove ARCH_AT91RM9200 option for drivers Nicolas Ferre
2014-11-27 16:06 ` [PATCH 05/11] ARM: at91: always USE_OF from now on Nicolas Ferre
2014-11-27 16:06 ` [PATCH 06/11] ARM: at91/trivial: update Kconfig comment to mention SAMA5 Nicolas Ferre
2014-11-27 16:06 ` [PATCH 07/11] ARM: at91: remove all !DT related configuration options Nicolas Ferre
2014-11-27 16:06 ` [PATCH 08/11] ARM: at91: remove clock data in at91sam9n12.c and at91sam9x5.c files Nicolas Ferre
2014-11-27 16:06 ` [PATCH 09/11] ARM: at91: remove old at91-specific clock driver Nicolas Ferre
2014-11-27 16:06 ` [PATCH 10/11] ARM: at91: remove legacy IRQ driver and related code Nicolas Ferre
2014-11-27 16:20   ` Julia Lawall
2014-11-27 16:06 ` [PATCH 11/11] ARM: at91: remove unused IRQ function declarations Nicolas Ferre
2014-11-27 16:49 ` [PATCH 00/11] ARM: at91: remove !DT support for at91rm9200 Arnd Bergmann
2014-11-27 17:12   ` Alexandre Belloni
2014-11-27 17:38     ` Arnd Bergmann
2014-11-28  9:36       ` Nicolas Ferre
2014-11-28 10:25         ` Arnd Bergmann
2014-11-28 10:48           ` Nicolas Ferre
2014-11-27 23:12     ` Arnd Bergmann
2014-11-27 23:39       ` Boris Brezillon [this message]
2014-11-27 23:41         ` Arnd Bergmann
2014-11-28  0:28       ` Alexandre Belloni
2014-11-28  8:27         ` Alexander Stein
2014-11-28  8:49           ` Arnd Bergmann
2014-11-28 11:31             ` Alexandre Belloni
2014-11-28 12:06               ` Arnd Bergmann
2014-11-28 11:13 ` Boris Brezillon

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=20141128003938.7efb4ea0@bbrezillon \
    --to=boris.brezillon@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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).