Re: [PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-02 Thread Benjamin Herrenschmidt
2009-02-02 14:16:04.0 +1100 @@ -599,8 +599,8 @@ int savage_driver_firstopen(struct drm_d drm_mtrr_add(dev_priv-mtrr[2].base, dev_priv-mtrr[2].size, DRM_MTRR_WC); } else { -

Re: [PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-02 Thread Benjamin Herrenschmidt
On Mon, 2009-02-02 at 11:02 -0800, Eric Anholt wrote: On Mon, 2009-02-02 at 16:55 +1100, Benjamin Herrenschmidt wrote: The DRM uses its own wrappers to obtain resources from PCI devices, which currently convert the resource_size_t into an unsigned long. This is broken on 32-bit platforms

Re: [PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-02 Thread Eric Anholt
On Mon, 2009-02-02 at 16:55 +1100, Benjamin Herrenschmidt wrote: The DRM uses its own wrappers to obtain resources from PCI devices, which currently convert the resource_size_t into an unsigned long. This is broken on 32-bit platforms with 32-bit physical address space. This fixes them,

Re: [PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-02 Thread Dave Airlie
On Mon, Feb 2, 2009 at 3:55 PM, Benjamin Herrenschmidt b...@kernel.crashing.org wrote: The DRM uses its own wrappers to obtain resources from PCI devices, which currently convert the resource_size_t into an unsigned long. This is broken on 32-bit platforms with 32-bit physical address space.

[PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-01 Thread Benjamin Herrenschmidt
The DRM uses its own wrappers to obtain resources from PCI devices, which currently convert the resource_size_t into an unsigned long. This is broken on 32-bit platforms with 32-bit physical address space. This fixes them, along with a few occurences of unsigned long used to store such a