> -----Original Message-----
> From: Gerd Hoffmann [mailto:kra...@redhat.com]
> Sent: Monday, February 24, 2014 5:17 AM
> To: linux-fb...@vger.kernel.org
> Cc: linux-ker...@vger.kernel.org; gre...@linuxfoundation.org; KY
> Srinivasan; Abhishek Gupta (LIS); Haiyang Zhang; jasow...@redhat.com;
> tomi.valkei...@ti.com; de...@linuxdriverproject.org; Gerd Hoffmann
> Subject: [PATCH v3 2/4] vmbus: use resource for hyperv mmio region
>
> Use a resource for the hyperv mmio region instead of start/size variables.
> Register the region properly so it shows up in /proc/iomem.
>
Thanks Gerd.
K. Y
> Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
Signed-off-by: K. Y. Srinivasan <k...@microsoft.com>
> ---
> drivers/hv/vmbus_drv.c | 16 ++++++++++------ include/linux/hyperv.h | 3
> +--
> 2 files changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c index
> 2352ae48..a14f603 100644
> --- a/drivers/hv/vmbus_drv.c
> +++ b/drivers/hv/vmbus_drv.c
> @@ -43,10 +43,12 @@ static struct acpi_device *hv_acpi_dev; static struct
> tasklet_struct msg_dpc; static struct completion probe_event; static int
> irq;
> -u64 hyperv_mmio_start;
> -EXPORT_SYMBOL_GPL(hyperv_mmio_start);
> -u64 hyperv_mmio_size;
> -EXPORT_SYMBOL_GPL(hyperv_mmio_size);
> +
> +struct resource hyperv_mmio = {
> + .name = "hyperv mmio",
> + .flags = IORESOURCE_MEM,
> +};
> +EXPORT_SYMBOL_GPL(hyperv_mmio);
>
> static int vmbus_exists(void)
> {
> @@ -902,8 +904,8 @@ static acpi_status vmbus_walk_resources(struct
> acpi_resource *res, void *ctx)
> break;
>
> case ACPI_RESOURCE_TYPE_ADDRESS64:
> - hyperv_mmio_start = res->data.address64.minimum;
> - hyperv_mmio_size = res->data.address64.address_length;
> + hyperv_mmio.start = res->data.address64.minimum;
> + hyperv_mmio.end = res->data.address64.maximum;
> break;
> }
>
> @@ -933,6 +935,8 @@ static int vmbus_acpi_add(struct acpi_device *device)
>
> if (ACPI_FAILURE(result))
> goto acpi_walk_err;
> + if (hyperv_mmio.start && hyperv_mmio.end)
> + request_resource(&iomem_resource,
> &hyperv_mmio);
> }
> ret_val = 0;
>
> diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index
> be3028f..c93e342 100644
> --- a/include/linux/hyperv.h
> +++ b/include/linux/hyperv.h
> @@ -1459,8 +1459,7 @@ int hv_vss_init(struct hv_util_service *); void
> hv_vss_deinit(void); void hv_vss_onchannelcallback(void *);
>
> -extern u64 hyperv_mmio_start;
> -extern u64 hyperv_mmio_size;
> +extern struct resource hyperv_mmio;
>
> /*
> * Negotiated version with the Host.
> --
> 1.8.3.1
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
