From: Paul Mundt on
On Tue, Mar 09, 2010 at 09:09:54AM +0100, Samuel Ortiz wrote:
>
> We should only request for the MFD used region, not the whole thing.
>
> Signed-off-by: Samuel Ortiz <sameo(a)linux.intel.com>

Works for me, thanks.

Tested-by: Paul Mundt <lethal(a)linux-sh.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: H Hartley Sweeten on
On Tuesday, March 09, 2010 1:10 AM, Samuel Ortiz wrote:
>
> We should only request for the MFD used region, not the whole thing.
>
> Signed-off-by: Samuel Ortiz <sameo(a)linux.intel.com>

Reviewed-by: H Hartley Sweeten <hsweeten(a)visionengravers.com>

Sorry about that...

> ---
> drivers/mfd/sm501.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mfd/sm501.c b/drivers/mfd/sm501.c
> index dc9ea95..497712f 100644
> --- a/drivers/mfd/sm501.c
> +++ b/drivers/mfd/sm501.c
> @@ -1430,7 +1430,7 @@ static int __devinit sm501_plat_probe(struct platform_device *dev)
> }
>
> sm->regs_claim = request_mem_region(sm->io_res->start,
> - resource_size(sm->io_res), "sm501");
> + 0x100, "sm501");
>
> if (sm->regs_claim == NULL) {
> dev_err(&dev->dev, "cannot claim registers\n");
> @@ -1644,7 +1644,7 @@ static int __devinit sm501_pci_probe(struct pci_dev *dev,
> sm->mem_res = &dev->resource[0];
>
> sm->regs_claim = request_mem_region(sm->io_res->start,
> - resource_size(sm->io_res), "sm501");
> + 0x100, "sm501");
> if (sm->regs_claim == NULL) {
> dev_err(&dev->dev, "cannot claim registers\n");
> err= -EBUSY;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/