diff mbox

ARM: shmobile: Enable Armadillo 800 EVA board in multiplatform defconfig

Message ID 1409229622-11566-1-git-send-email-geert+renesas@glider.be (mailing list archive)
State New, archived
Headers show

Commit Message

Geert Uytterhoeven Aug. 28, 2014, 12:40 p.m. UTC
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/configs/shmobile_defconfig | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Arnd Bergmann Aug. 28, 2014, 12:49 p.m. UTC | #1
On Thursday 28 August 2014 14:40:22 Geert Uytterhoeven wrote:
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  arch/arm/configs/shmobile_defconfig | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 

No objections to the patch, but I wonder if you could send another
patch to enable all the machines working in ARCH_SHMOBILE_MULTI
as part of multi_v7_defconfig as well.

	Arnd
Geert Uytterhoeven Aug. 28, 2014, 1:06 p.m. UTC | #2
Hi Arnd,

On Thu, Aug 28, 2014 at 2:49 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Thursday 28 August 2014 14:40:22 Geert Uytterhoeven wrote:
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>>  arch/arm/configs/shmobile_defconfig | 12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>
> No objections to the patch, but I wonder if you could send another
> patch to enable all the machines working in ARCH_SHMOBILE_MULTI
> as part of multi_v7_defconfig as well.

Sure. I actually have such a patch lying around :-)

I've just added armadillo support there, too, but it hangs during boot
on Armadillo. May be that the image is just too big for the Hermit boot
loader.
It does work on Koelsch. Haven't tried it yet on BeagleBone Black, though.

Do you want it? Or do you want the version without armadillo support?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Arnd Bergmann Aug. 28, 2014, 1:24 p.m. UTC | #3
On Thursday 28 August 2014 15:06:31 Geert Uytterhoeven wrote:
> On Thu, Aug 28, 2014 at 2:49 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > On Thursday 28 August 2014 14:40:22 Geert Uytterhoeven wrote:
> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> ---
> >>  arch/arm/configs/shmobile_defconfig | 12 ++++++++++++
> >>  1 file changed, 12 insertions(+)
> >
> > No objections to the patch, but I wonder if you could send another
> > patch to enable all the machines working in ARCH_SHMOBILE_MULTI
> > as part of multi_v7_defconfig as well.
> 
> Sure. I actually have such a patch lying around 
> 
> I've just added armadillo support there, too, but it hangs during boot
> on Armadillo. May be that the image is just too big for the Hermit boot
> loader.
> It does work on Koelsch. Haven't tried it yet on BeagleBone Black, though.
> 
> Do you want it? Or do you want the version without armadillo support?
> 

Good question. Ideally I'd like them all to be enabled, but I guess there
is little value in adding Armadillo if it doesn't actually work.

Maybe you have run into the same u-boot problem that a few other people
had before? When mach-qcom is enabled, the kernel entry point changes
and you have to adapt any script that tries to build a uImage manually.

	Arnd
Geert Uytterhoeven Aug. 28, 2014, 1:39 p.m. UTC | #4
Hi Arnd,

On Thu, Aug 28, 2014 at 3:24 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Thursday 28 August 2014 15:06:31 Geert Uytterhoeven wrote:
>> On Thu, Aug 28, 2014 at 2:49 PM, Arnd Bergmann <arnd@arndb.de> wrote:
>> > No objections to the patch, but I wonder if you could send another
>> > patch to enable all the machines working in ARCH_SHMOBILE_MULTI
>> > as part of multi_v7_defconfig as well.
>>
>> Sure. I actually have such a patch lying around
>>
>> I've just added armadillo support there, too, but it hangs during boot
>> on Armadillo. May be that the image is just too big for the Hermit boot
>> loader.
>> It does work on Koelsch. Haven't tried it yet on BeagleBone Black, though.
>>
>> Do you want it? Or do you want the version without armadillo support?
>
> Good question. Ideally I'd like them all to be enabled, but I guess there
> is little value in adding Armadillo if it doesn't actually work.
>
> Maybe you have run into the same u-boot problem that a few other people
> had before? When mach-qcom is enabled, the kernel entry point changes
> and you have to adapt any script that tries to build a uImage manually.

Armadillo uses hermit, not u-boot, and boots a zImage with appended DTB.
I tried disabling CONFIG_ARCH_QCOM, but it didn't make a difference.
It does boot the shmobile_defconfig kernel, which is only 3 MiB large
(compared to 5.4 for multi_v7_defconfig).

For now, I'll send you the version without armadillo support.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Simon Horman Sept. 1, 2014, 1:47 a.m. UTC | #5
On Thu, Aug 28, 2014 at 02:40:22PM +0200, Geert Uytterhoeven wrote:
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, I have queued this up.
diff mbox

Patch

diff --git a/arch/arm/configs/shmobile_defconfig b/arch/arm/configs/shmobile_defconfig
index fa94612d72cc..d3bced4555aa 100644
--- a/arch/arm/configs/shmobile_defconfig
+++ b/arch/arm/configs/shmobile_defconfig
@@ -13,6 +13,7 @@  CONFIG_SLAB=y
 CONFIG_ARCH_SHMOBILE_MULTI=y
 CONFIG_ARCH_EMEV2=y
 CONFIG_ARCH_R7S72100=y
+CONFIG_ARCH_R8A7740=y
 CONFIG_ARCH_R8A7779=y
 CONFIG_ARCH_R8A7790=y
 CONFIG_ARCH_R8A7791=y
@@ -75,6 +76,8 @@  CONFIG_SMSC_PHY=y
 # CONFIG_INPUT_MOUSEDEV_PSAUX is not set
 CONFIG_KEYBOARD_GPIO=y
 # CONFIG_INPUT_MOUSE is not set
+CONFIG_INPUT_TOUCHSCREEN=y
+CONFIG_TOUCHSCREEN_ST1232=y
 # CONFIG_LEGACY_PTYS is not set
 CONFIG_SERIAL_8250=y
 CONFIG_SERIAL_8250_CONSOLE=y
@@ -113,10 +116,16 @@  CONFIG_VIDEO_RENESAS_VSP1=y
 CONFIG_VIDEO_ADV7180=y
 CONFIG_DRM=y
 CONFIG_DRM_RCAR_DU=y
+CONFIG_BACKLIGHT_LCD_SUPPORT=y
+# CONFIG_LCD_CLASS_DEVICE is not set
+# CONFIG_BACKLIGHT_GENERIC is not set
+CONFIG_BACKLIGHT_PWM=y
 CONFIG_SOUND=y
 CONFIG_SND=y
 CONFIG_SND_SOC=y
+CONFIG_SND_SOC_SH4_FSI=y
 CONFIG_SND_SOC_RCAR=y
+CONFIG_SND_SOC_WM8978=y
 CONFIG_USB=y
 CONFIG_USB_EHCI_HCD=y
 CONFIG_USB_OHCI_HCD=y
@@ -133,9 +142,12 @@  CONFIG_NEW_LEDS=y
 CONFIG_LEDS_CLASS=y
 CONFIG_LEDS_GPIO=y
 CONFIG_RTC_CLASS=y
+CONFIG_RTC_DRV_S35390A=y
 CONFIG_DMADEVICES=y
 CONFIG_SH_DMAE=y
 # CONFIG_IOMMU_SUPPORT is not set
+CONFIG_PWM=y
+CONFIG_PWM_RENESAS_TPU=y
 # CONFIG_DNOTIFY is not set
 CONFIG_MSDOS_FS=y
 CONFIG_VFAT_FS=y