public inbox for linux-arm-kernel@lists.infradead.org 
 help / color / mirror / Atom feed
From: luka@openwrt•org (Luka Perkov)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH] IXP4xx: Add Gateworks Cambria support.
Date: Thu, 19 Sep 2013 18:03:09 +0200	[thread overview]
Message-ID: <20130919160309-31798@mutt-kz> (raw)
In-Reply-To: <m3vc1ya9fw.fsf@t19.piap.pl>

Hi Krzysztof,

it's great that you have decided to push upstream support for this
board.

I was unable to bring up serial console with your patch. I was using
latest OpenWrt trunk for rootfs, did you use the same?

drivers/rtc/hctosys.c: unable to open rtc device (rtc0)
IP-Config: Failed to open eth0
IP-Config: Failed to open eth1
IP-Config: No network devices available
Warning: unable to open an initial console.
Freeing unused kernel memory: 4124K (c0427000 - c082e000)
procd: Console is alive

On Wed, Sep 18, 2013 at 09:13:55AM +0200, Krzysztof Ha?asa wrote:
> --- /dev/null
> +++ b/arch/arm/mach-ixp4xx/cambria.c
> @@ -0,0 +1,245 @@
> +/*
> + * Gateworks Cambria GW2350 platform support
> + * Portions based on OpenWRT.
> + */

I'm no expert on licenses but shouldn't here at least be GPLv2 since the
file originated from OpenWrt support for this board which is GPLv2.

> +static struct i2c_board_info __initdata cambria_i2c_board_info[] = {
> +	{
> +		I2C_BOARD_INFO("ad7418", 0x28),
> +	},
> +	{
> +		I2C_BOARD_INFO("24c08", 0x50),
> +		.platform_data	= &cambria_eeprom_info,
> +	},
> +};

I couldn't test this because of the console issue but in OpenWrt sources
the value is 0x51 and not 0x50. Did you change it by accident?

Also, shouldn't arch/arm/configs/ixp4xx_defconfig be updated too?

Luka

  reply	other threads:[~2013-09-19 16:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-18  7:13 [PATCH] IXP4xx: Add Gateworks Cambria support Krzysztof Hałasa
2013-09-19 16:03 ` Luka Perkov [this message]
2013-09-19 18:27   ` Krzysztof Halasa
2013-09-19 20:52 ` Imre Kaloz
2013-09-20  8:40   ` Krzysztof Hałasa
2013-09-20  9:13     ` Imre Kaloz
2013-09-25  7:24       ` Krzysztof Hałasa

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=20130919160309-31798@mutt-kz \
    --to=luka@openwrt$(echo .)org \
    --cc=linux-arm-kernel@lists$(echo .)infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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