lkml.org 
[lkml]   [2012]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] iommu/tegra: gart: Reserve iomem resource
* Hiroshi DOYU wrote:
> GART Register/Aperture range should be reserved.
>
> Signed-off-by: Hiroshi DOYU <hdoyu@nvidia.com>
> ---
> drivers/iommu/tegra-gart.c | 8 +++++++-
> 1 files changed, 7 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/iommu/tegra-gart.c b/drivers/iommu/tegra-gart.c
> index 33a23b6..a431006 100644
> --- a/drivers/iommu/tegra-gart.c
> +++ b/drivers/iommu/tegra-gart.c
> @@ -366,12 +366,18 @@ static int tegra_gart_probe(struct platform_device *pdev)
> return -ENOMEM;
> }
>
> - gart_regs = devm_ioremap(dev, res->start, resource_size(res));
> + gart_regs = devm_request_and_ioremap(dev, res);
> if (!gart_regs) {
> dev_err(dev, "failed to remap GART registers\n");
> return -ENXIO;
> }
>
> + if (!devm_request_mem_region(dev, res_remap->start,
> + resource_size(res_remap), dev_name(dev))) {
> + dev_err(dev, "failed to reserve GART aperture\n");
> + return -EBUSY;
> + }
> +

I'm not so sure that this is a good idea. If the GART driver requests the
GART aperture, then users of the GART aperture (e.g. the DRM driver) will no
longer be able to use it.

Thierry
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2012-05-21 17:01    [W:0.060 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site