Re: [PATCH 1/2] platform/x86: x86-android-tablets: Fix Bluetooth on Lenovo Yoga Book

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 4/29/23 20:02, Hans de Goede wrote:
> The Lenovo Yoga Book yb1-x90f/l has (another) bug in its DSDT where
> the UART resource for the BTH0 ACPI device contains
> "\\_SB.PCIO.URT1" as path to the UART.
> 
> Note that is with a letter 'O' instead of the number '0' which is wrong.
> 
> Add a x86_serdev_info entry to make the x86-android-tablets module
> manually setup the /sys/bus/serial device for the Bluetooth UART
> to fix Bluetooth not working due to this bug.
> 
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

I've added this series to my review-hans (soon to be for-next) branch now.

Regards,

Hans




> ---
>  drivers/platform/x86/x86-android-tablets/lenovo.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/drivers/platform/x86/x86-android-tablets/lenovo.c b/drivers/platform/x86/x86-android-tablets/lenovo.c
> index 0297b4c43d3b..270c4712fa7c 100644
> --- a/drivers/platform/x86/x86-android-tablets/lenovo.c
> +++ b/drivers/platform/x86/x86-android-tablets/lenovo.c
> @@ -147,6 +147,19 @@ static const struct platform_device_info lenovo_yb1_x90_pdevs[] __initconst = {
>  	},
>  };
>  
> +/*
> + * DSDT says UART path is "\\_SB.PCIO.URT1" with a letter 'O' instead of
> + * the number '0' add the link manually.
> + */
> +static const struct x86_serdev_info lenovo_yb1_x90_serdevs[] __initconst = {
> +	{
> +		.ctrl_hid = "8086228A",
> +		.ctrl_uid = "1",
> +		.ctrl_devname = "serial0",
> +		.serdev_hid = "BCM2E1A",
> +	},
> +};
> +
>  static struct gpiod_lookup_table lenovo_yb1_x90_goodix_gpios = {
>  	.dev_id = "i2c-goodix_ts",
>  	.table = {
> @@ -203,6 +216,8 @@ const struct x86_dev_info lenovo_yogabook_x90_info __initconst = {
>  	.i2c_client_count = ARRAY_SIZE(lenovo_yb1_x90_i2c_clients),
>  	.pdev_info = lenovo_yb1_x90_pdevs,
>  	.pdev_count = ARRAY_SIZE(lenovo_yb1_x90_pdevs),
> +	.serdev_info = lenovo_yb1_x90_serdevs,
> +	.serdev_count = ARRAY_SIZE(lenovo_yb1_x90_serdevs),
>  	.gpiod_lookup_tables = lenovo_yb1_x90_gpios,
>  	.init = lenovo_yb1_x90_init,
>  };