All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arc: add stubs for map_physmem() and unmap_physmem()
Date: Thu, 12 Nov 2015 16:00:45 -0700	[thread overview]
Message-ID: <56451A1D.3080104@wwwdotorg.org> (raw)
In-Reply-To: <1447365381-16342-1-git-send-email-abrodkin@synopsys.com>

On 11/12/2015 02:56 PM, Alexey Brodkin wrote:
> Up until now there was no need in those stubs.
>
> But since following commit compilation of U-Boot on ARC is broken:
> commit 7861204c9af7fec1ea9b41541c272516235a6c93
>      itest: make memory access work under sandbox
>
...
> That's because CMD_ITEST is enabled by default in common/Kconfig and now
> map_physmem()/unmap_physmem() is used there.
>
> So this patch adds missing stubs for ARC.

This looks OK, but rather than cut/pasting this exact same code yet 
another time, why not create e.g. include/io-base.h that contains this, 
and share it amongst all architectures?

  reply	other threads:[~2015-11-12 23:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-12 21:56 [U-Boot] [PATCH] arc: add stubs for map_physmem() and unmap_physmem() Alexey Brodkin
2015-11-12 23:00 ` Stephen Warren [this message]
2015-11-13 13:40   ` Alexey Brodkin
2015-11-13 16:06     ` Stephen Warren
2015-11-13 18:14 ` Simon Glass
2015-11-13 18:23   ` Alexey Brodkin
2015-11-14  2:03     ` Simon Glass
2015-11-16 13:47       ` Alexey Brodkin
2015-11-16 21:08         ` Simon Glass
2015-11-16 22:15           ` Alexey Brodkin
2015-11-16 23:15             ` Stephen Warren
2015-11-17 21:43               ` Alexey Brodkin

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=56451A1D.3080104@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=u-boot@lists.denx.de \
    /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.