All of lore.kernel.org
 help / color / mirror / Atom feed
From: jason77.wang@gmail.com (jason)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] mxc: add common debug board for 3-stack platforms
Date: Fri, 18 Jun 2010 22:06:34 +0800	[thread overview]
Message-ID: <4C1B7D6A.4010301@gmail.com> (raw)
In-Reply-To: <20100616065540.GI20799@pengutronix.de>

Sascha Hauer wrote:
> On Sat, Jun 12, 2010 at 10:25:06PM +0800, Jason Wang wrote:
>   
>> The debug board is little different for all mxc 3-stack(PDK)
>> platforms, it is possible here to add a common implementation to
>> support this board.
>>     
>
> Two small things left:
>
> Could you change the name cpld_debugboard.c to something like
> 3ds_debugboard.c? That we are talking about the 3ds is much more
> interesting than that there's a cpld on the board.
>
> Also, I really prefer having no extra header file for the debug board.
> All information in this header file is only interesting in this single C
> file.
>
> Sascha
>
>
>   
Hi Sascha,

About the header file, most contents can be moved to c file,
but there are two declarations:

extern struct platform_device smsc_lan9217_device;
int __init mxc_expio_init(u32 base, u32 p_irq);

which will be used by board_mxNN_3ds.c, should i move them to mach/common.h and
drop 3ds_debugboard.h or keep them in 3ds_debugboard.h?

Thanks,
Jason.

>> Signed-off-by: Jason Wang <jason77.wang@gmail.com>
>> ---
>>  arch/arm/plat-mxc/Kconfig                        |   11 ++
>>  arch/arm/plat-mxc/Makefile                       |    1 +
>>  arch/arm/plat-mxc/cpld_debugboard.c              |  161 ++++++++++++++++++++++
>>     
<snip>

  parent reply	other threads:[~2010-06-18 14:06 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-12 14:25 [PATCH 0/2]Add mxc common CPLD debugboard support Jason Wang
2010-06-12 14:25 ` [PATCH 1/2] mxc: add common debug board for 3-stack platforms Jason Wang
2010-06-12 14:25   ` [PATCH 2/2] mx31_3ds: revert cpld debug board relate code Jason Wang
2010-06-16  6:57     ` Sascha Hauer
2010-06-16 20:36       ` Magnus Lilja
2010-06-16  6:55   ` [PATCH 1/2] mxc: add common debug board for 3-stack platforms Sascha Hauer
2010-06-17 13:37     ` jason
2010-06-18 14:06     ` jason [this message]
2010-06-21  7:11       ` Sascha Hauer
2010-06-22 13:08         ` jason

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=4C1B7D6A.4010301@gmail.com \
    --to=jason77.wang@gmail.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.