mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* Regression on latest barebox
@ 2012-10-08  9:23 Alexander Shiyan
       [not found] ` <20121008094508.GN1322@pengutronix.de>
  0 siblings, 1 reply; 2+ messages in thread
From: Alexander Shiyan @ 2012-10-08  9:23 UTC (permalink / raw)
  To: barebox

Hello.

Latest master sources have a regression due migrate to use platform_device_register functions.
Now I cannot register cfi_flash device which resides at 0x0, because it have conflicts with "mem"
command which have same start address.
Thanks!

---
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re[2]: Regression on latest barebox
       [not found] ` <20121008094508.GN1322@pengutronix.de>
@ 2012-10-08  9:58   ` Alexander Shiyan
  0 siblings, 0 replies; 2+ messages in thread
From: Alexander Shiyan @ 2012-10-08  9:58 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox

Mon, 8 Oct 2012 11:45:08 +0200 от Sascha Hauer <s.hauer@pengutronix.de>:

> On Mon, Oct 08, 2012 at 01:23:54PM +0400, Alexander Shiyan wrote:
> > Hello.
> > Latest master sources have a regression due migrate to use platform_device_register functions.
> > Now I cannot register cfi_flash device which resides at 0x0, because it have conflicts with "mem"
> > command which have same start address.
> 
> This is because of this quite bogus check in platform_device_register:
> 
> 	if (dev->resource->start == new_device->resource->start) {
> 		return -EBUSY;
> 	}
> 
> The correct way would be to check whether the regions overlap, not
> whether the start addresses match. With this the whole patch wouldn't
> work anymore because everything would conflict with the iomem resource.
> 
> I have no good idea atm except making /dev/mem a non driver.
Or we can provide virtual bus ("system" for example) for /dev/mem device.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-08  9:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-08  9:23 Regression on latest barebox Alexander Shiyan
     [not found] ` <20121008094508.GN1322@pengutronix.de>
2012-10-08  9:58   ` Re[2]: " Alexander Shiyan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox