ACK/Cmnt: [PATCH 1/1][SRU][D][OEM-OSP1-B] UBUNTU: SAUCE: i2c: designware: add Inpiron 7591 into i2c quirk

[Stefan Bader]

On 11.07.19 10:58, AceLan Kao wrote:
> BugLink: https://bugs.launchpad.net/bugs/1836020
> 
> Add one missing platform which uses Goodix touchpad IC into i2c designware
> quirk to prevent touchpad from entering runtime suspend.
> 
> Signed-off-by: AceLan Kao <acelan.kao at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>
> ---

The way this is submitted is confusing and somewhat prone to errors. The patch
depends on the other addition to the quirks for the 7590 but as it is submitted
independently it could get picked up before the other one. I only looked closer
because the status of the bug report already is fix-committed which sounded wrong.

In this case I think it would be better to submit a follow-up v2 which carries
both patches and self-NACK the initial one. Or, if the first patch already is
applied, then follow-up with the additional patch on the initial thread.

-Stefan

>  drivers/i2c/busses/i2c-designware-master.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/i2c/busses/i2c-designware-master.c b/drivers/i2c/busses/i2c-designware-master.c
> index 00a906ec43be..d1063ef81ac6 100644
> --- a/drivers/i2c/busses/i2c-designware-master.c
> +++ b/drivers/i2c/busses/i2c-designware-master.c
> @@ -46,6 +46,13 @@ static const struct dmi_system_id i2c_dw_no_runtime_pm[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 7590"),
>  		},
>  	},
> +	{
> +		.ident = "Dell Inspiron 7591",
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 7591"),
> +		},
> +	},
>  	{
>  		.ident = "Dell Vostro 5390",
>  		.matches = {
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20190719/cb7b89f6/attachment-0001.sig>