2011-05-25 16:27:40

by Linus Walleij

Subject: [PATCH 0/2] Support for the ST-Ericsson HSI controller

From: Linus Walleij <[email protected]>

Hi Carlos,

here is the pending patch to add support for ST-Ericssons HSI controller
in your new HSI subsystem.

It would be great if you could carry this as part of the HSI patchset until
it gets merged so you have full control over it, but I think we need to
split out the <mash/hsi.h> file from the second patch and put it into
the hsi include dir first for that.

Anyway, take it as a first sign of life and a review opportunity.

Pawel Szyszuk (2):
HSI: add ST-E HSI controller
mach-ux500: platform data for ST-E HSI controller

arch/arm/mach-ux500/board-mop500-pins.c | 10 +
arch/arm/mach-ux500/board-mop500.c | 65 ++
arch/arm/mach-ux500/clock.c | 8 +-
arch/arm/mach-ux500/devices-db8500.c | 118 +++
arch/arm/mach-ux500/include/mach/devices.h | 1 +
arch/arm/mach-ux500/include/mach/hsi.h | 122 +++
drivers/hsi/controllers/Kconfig | 10 +
drivers/hsi/controllers/Makefile | 1 +
drivers/hsi/controllers/ste_hsi.c | 1536 ++++++++++++++++++++++++++++
9 files changed, 1867 insertions(+), 4 deletions(-)
create mode 100644 arch/arm/mach-ux500/include/mach/hsi.h
create mode 100644 drivers/hsi/controllers/ste_hsi.c

--
1.7.3.2

2011-05-30 08:54:14

by Carlos Chinea

Subject: Re: [PATCH 0/2] Support for the ST-Ericsson HSI controller

Hi Linus,

On Wed, 2011-05-25 at 18:27 +0200, ext Linus Walleij wrote:
> From: Linus Walleij <[email protected]>
>
> Hi Carlos,
>
> here is the pending patch to add support for ST-Ericssons HSI controller
> in your new HSI subsystem.
>
> It would be great if you could carry this as part of the HSI patchset until
> it gets merged so you have full control over it, but I think we need to
> split out the <mash/hsi.h> file from the second patch and put it into
> the hsi include dir first for that.
>

I will be glad to carry this patches with the HSI patchset. I'll do a
review asap. As first comment I think we can keep mach/hsi.h where it is
as it looks to me that all the code is tight there is tied to the
platform.

> Anyway, take it as a first sign of life and a review opportunity.
>
> Pawel Szyszuk (2):
> HSI: add ST-E HSI controller
> mach-ux500: platform data for ST-E HSI controller
>
> arch/arm/mach-ux500/board-mop500-pins.c | 10 +
> arch/arm/mach-ux500/board-mop500.c | 65 ++
> arch/arm/mach-ux500/clock.c | 8 +-
> arch/arm/mach-ux500/devices-db8500.c | 118 +++
> arch/arm/mach-ux500/include/mach/devices.h | 1 +
> arch/arm/mach-ux500/include/mach/hsi.h | 122 +++
> drivers/hsi/controllers/Kconfig | 10 +
> drivers/hsi/controllers/Makefile | 1 +
> drivers/hsi/controllers/ste_hsi.c | 1536 ++++++++++++++++++++++++++++
> 9 files changed, 1867 insertions(+), 4 deletions(-)
> create mode 100644 arch/arm/mach-ux500/include/mach/hsi.h
> create mode 100644 drivers/hsi/controllers/ste_hsi.c
>

Br,
Carlos

--
Carlos Chinea <[email protected]>