On Tue, Nov 08, 2022 at 04:22:24PM +0200, Andy Shevchenko wrote:
> The PWM LPSS device can be embedded in another device.
> In order to enable it, allow that drivers to probe
> a corresponding device.
"probe" isn't the right term here. The other driver only creates the
device.
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> ---
> drivers/pwm/pwm-lpss.h | 22 +--------------
> .../linux/platform_data/x86}/pwm-lpss.h | 28 ++++---------------
> 2 files changed, 6 insertions(+), 44 deletions(-)
> copy {drivers/pwm => include/linux/platform_data/x86}/pwm-lpss.h (53%)
>
> diff --git a/drivers/pwm/pwm-lpss.h b/drivers/pwm/pwm-lpss.h
> index 4561d229b27d..b721532c6c3c 100644
> --- a/drivers/pwm/pwm-lpss.h
> +++ b/drivers/pwm/pwm-lpss.h
> @@ -13,27 +13,10 @@
> #include <linux/pwm.h>
> #include <linux/types.h>
>
> -struct device;
> +#include <linux/platform_data/x86/pwm-lpss.h>
>
> #define LPSS_MAX_PWMS 4
>
> -struct pwm_lpss_boardinfo {
> - unsigned long clk_rate;
> - unsigned int npwm;
> - unsigned long base_unit_bits;
> - /*
> - * Some versions of the IP may stuck in the state machine if enable
> - * bit is not set, and hence update bit will show busy status till
> - * the reset. For the rest it may be otherwise.
> - */
> - bool bypass;
> - /*
> - * On some devices the _PS0/_PS3 AML code of the GPU (GFX0) device
> - * messes with the PWM0 controllers state,
> - */
> - bool other_devices_aml_touches_pwm_regs;
> -};
> -
Now that pwm_lpss_boardinfo lives in a different file, this makes the
move of pwm_lpss_chip in patch 3 somewhat redundant.
> extern const struct pwm_lpss_boardinfo pwm_lpss_byt_info;
> extern const struct pwm_lpss_boardinfo pwm_lpss_bsw_info;
> extern const struct pwm_lpss_boardinfo pwm_lpss_bxt_info;
> @@ -45,7 +28,4 @@ struct pwm_lpss_chip {
> const struct pwm_lpss_boardinfo *info;
> };
>
> -struct pwm_lpss_chip *pwm_lpss_probe(struct device *dev, void __iomem *base,
> - const struct pwm_lpss_boardinfo *info);
> -
> #endif /* __PWM_LPSS_H */
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
Attachment:
signature.asc
Description: PGP signature
