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
WARNING: multiple messages have this Message-ID (diff)
From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
Alexandre Belloni <alexandre.belloni@free-electrons.com>,
Nicolas Ferre <nicolas.ferre@atmel.com>,
linux-kernel@vger.kernel.org,
Ludovic Desroches <ludovic.desroches@atmel.com>,
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Subject: Re: [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
next prev parent reply other threads:[~2014-11-27 23:39 UTC|newest]
Thread overview: 56+ 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 ` Nicolas Ferre
2014-11-27 16:06 ` [PATCH 01/11] ARM: at91: remove at91rm9200 legacy boards files Nicolas Ferre
2014-11-27 16:06 ` Nicolas Ferre
2014-11-27 16:06 ` [PATCH 02/11] ARM: at91: remove at91rm9200 legacy board support Nicolas Ferre
2014-11-27 16:06 ` 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 ` 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 ` 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 ` 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 ` 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 ` 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 ` 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 ` 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:06 ` Nicolas Ferre
2014-11-27 16:20 ` Julia Lawall
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:06 ` Nicolas Ferre
2014-11-27 16:49 ` [PATCH 00/11] ARM: at91: remove !DT support for at91rm9200 Arnd Bergmann
2014-11-27 16:49 ` Arnd Bergmann
2014-11-27 17:12 ` Alexandre Belloni
2014-11-27 17:12 ` Alexandre Belloni
2014-11-27 17:38 ` Arnd Bergmann
2014-11-27 17:38 ` Arnd Bergmann
2014-11-28 9:36 ` Nicolas Ferre
2014-11-28 9:36 ` Nicolas Ferre
2014-11-28 10:25 ` Arnd Bergmann
2014-11-28 10:25 ` Arnd Bergmann
2014-11-28 10:48 ` Nicolas Ferre
2014-11-28 10:48 ` Nicolas Ferre
2014-11-27 23:12 ` Arnd Bergmann
2014-11-27 23:12 ` Arnd Bergmann
2014-11-27 23:39 ` Boris Brezillon [this message]
2014-11-27 23:39 ` Boris Brezillon
2014-11-27 23:41 ` Arnd Bergmann
2014-11-27 23:41 ` Arnd Bergmann
2014-11-28 0:28 ` Alexandre Belloni
2014-11-28 0:28 ` Alexandre Belloni
2014-11-28 8:27 ` Alexander Stein
2014-11-28 8:27 ` Alexander Stein
2014-11-28 8:49 ` Arnd Bergmann
2014-11-28 8:49 ` Arnd Bergmann
2014-11-28 11:31 ` Alexandre Belloni
2014-11-28 11:31 ` Alexandre Belloni
2014-11-28 12:06 ` Arnd Bergmann
2014-11-28 12:06 ` Arnd Bergmann
2014-11-28 11:13 ` Boris Brezillon
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 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.