Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1200736

Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call.

Show all headers | View raw

On Tuesday, August 04, 2015 04:01:59 PM David Daney wrote:
> From: Tomasz Nowicki <tomasz.nowicki@linaro.org>
> 
> Fixes the following build error when building drivers as modules:
> 
>  ERROR: "acpi_dev_prop_read_single" [drivers/net/phy/mdio-octeon.ko] undefined!
>  ERROR: "acpi_dev_prop_read_single" [drivers/net/ethernet/cavium/thunder/thunder_bgx.ko] undefined!

Can you please tell me why the drivers in question use that function
directly, although they aren't supposed to?

Clearly, their authors had not tried to build them as modules or they
would have noticed the problem at the development stage already.

What would be wrong with using the generic device properties API instead?


> Reported-by: Andreas Schwab <schwab@suse.de>
> Signed-off-by: Tomasz Nowicki <tomasz.nowicki@linaro.org>
> Signed-off-by: Robert Richter <rrichter@cavium.com>
> Signed-off-by: David Daney <david.daney@cavium.com>
> ---
>  drivers/acpi/property.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/acpi/property.c b/drivers/acpi/property.c
> index 7836e2e..237e3c5 100644
> --- a/drivers/acpi/property.c
> +++ b/drivers/acpi/property.c
> @@ -432,6 +432,7 @@ int acpi_dev_prop_read_single(struct acpi_device *adev, const char *propname,
>  	}
>  	return ret;
>  }
> +EXPORT_SYMBOL_GPL(acpi_dev_prop_read_single);
>  
>  static int acpi_copy_property_array_u8(const union acpi_object *items, u8 *val,
>  				       size_t nval)
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] acpi, property: Export acpi_dev_prop_read_single call. David Daney <ddaney.cavm@gmail.com> - 2015-08-05 01:10 +0200
  Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-08-05 15:30 +0200
    Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. Tomasz Nowicki <tomasz.nowicki@linaro.org> - 2015-08-05 15:50 +0200
      Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. David Daney <ddaney@caviumnetworks.com> - 2015-08-05 19:30 +0200
        Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. David Daney <ddaney.cavm@gmail.com> - 2015-08-05 22:20 +0200
          Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-08-06 01:00 +0200
            Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-08-06 01:20 +0200
            Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2015-08-06 01:30 +0200
              Re: [PATCH] acpi, property: Export acpi_dev_prop_read_single call. David Daney <ddaney.cavm@gmail.com> - 2015-08-06 02:00 +0200

csiph-web