Hi, On 8/29/21 2:07 PM, Matan Ziv-Av wrote: > > The laptop model is identified by parsing the product name. If no > product name is available, do not try to parse it. > Default model is 2017. > > Signed-off-by: Matan Ziv-Av <matan@xxxxxxxxxxx> I will also add this to the fixes branch, so that it gets send out to Linus in my next pull-request with fixes for 5.15. Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans I will also add this to the fixes branch, so that it gets send out to Linus in my next pull-request with fixes for 5.15. Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/x86/lg-laptop.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/platform/x86/lg-laptop.c b/drivers/platform/x86/lg-laptop.c > index 3e520d5bca07..88b551caeaaf 100644 > --- a/drivers/platform/x86/lg-laptop.c > +++ b/drivers/platform/x86/lg-laptop.c > @@ -655,7 +655,7 @@ static int acpi_add(struct acpi_device *device) > goto out_platform_registered; > } > product = dmi_get_system_info(DMI_PRODUCT_NAME); > - if (strlen(product) > 4) > + if (product && strlen(product) > 4) > switch (product[4]) { > case '5': > case '6': >
