Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
From:   Li Jun <jun.li@nxp.com>
To:     myungjoo.ham@samsung.com, cw00.choi@samsung.com, krzk@kernel.org
Cc:     linux-kernel@vger.kernel.org, frank.li@nxp.com, xu.yang_2@nxp.com
Subject: [PATCH v2 1/2] extcon: ptn5150: add queue work sync before driver release
Date:   Tue, 19 Apr 2022 15:43:21 +0800
Message-Id: <1650354202-10445-1-git-send-email-jun.li@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Tj5lZhUqit+880vWsL7sMGMQLNmShHdwib6BU0z8UeZ414EhjBfsTj0oLt9v?=
 =?us-ascii?Q?4bddH1h7p8Gvqh2fBjprP3qyxV0vxRaiTfMqqL649FjwjEXBArByEdmwuEZ/?=
 =?us-ascii?Q?76pCx/dKxiZjyEy7p/73GzFiA1nKNMRUUW3nOEJeffgwz0p5v3GwzXjBbPJh?=
 =?us-ascii?Q?AMlVvn8bNVzLJ/JPEWgt1ZZM49tGf7fzEpNk+WznvEgJ0Cc1Kejm9WgXgG6Z?=
 =?us-ascii?Q?4PxSdG9OED+jabfPuHDvpSWmArDyjsRK5enh4zLaQeK6Gq5BmCmgz/zNN4fJ?=
 =?us-ascii?Q?BmQYiiUGHBgPTnG6rbPvwJqv0TlNAQZatH9VfI4gyO4QD2dDBybioCD2Uj66?=
 =?us-ascii?Q?LaIDMGgahH+Lz9YZWm5Qc7+lOcBV4g7bGYWJpjEpOlY3PFdqSkbz7K1rNu0h?=
 =?us-ascii?Q?88lo41D6J+OmlN3GBkJpxSKLEllHgnY60XQkYVRkGYGZoPhSSbDuYlvoAiyt?=
 =?us-ascii?Q?Igd1vRa3BTVz/KsToshlRG9Z3YRGm5Zz4OaV/pIgn72s5dALL8Q3UBbDpVIC?=
 =?us-ascii?Q?Tm3xfz2j6T+UjgpRjm6z/ITxcxaL/eHuZros3VtomAonpqHAqwI9GMhQ42gB?=
 =?us-ascii?Q?ERQz9F/Jsx3Y8osdrg6HnHTtDkua1sw9nzCUYEj83Z9aQPuDpgeIfuU6PaqU?=
 =?us-ascii?Q?FBzSs7YYEGkPMYsFUkKDtK6z5YeZuYs5Rp2bfcKH3trd6gYqOcg+tuaOqVNl?=
 =?us-ascii?Q?nbtj4GiwfezfTXSAxDyZ+Y/G+nrsIFz8vPWpjwnHO3nQAGFwHdcYAzNt/bP7?=
 =?us-ascii?Q?RffXKDLIly7FS0TJH1sNDroMRxOCHAa3PzH3O4GAmjF9J2QUcDFXz+QfxmP7?=
 =?us-ascii?Q?FZTdeu/iXxTxweGQ0EgstjJeMNCg5nyxgqSzJDf8e49MBWHps5rcu4rAlAAZ?=
 =?us-ascii?Q?pPgN59AvIHymjDNiVJ2Ib+6iVEq5goWi4R2knTKkspnz5sUq5T5k9Favixfw?=
 =?us-ascii?Q?ZZ6MbaxeOW8OAPgocF54BOCO9pYg7pky5szLYUPbdRnUalHS+m3RYvaoutrh?=
 =?us-ascii?Q?2WYQCfqyDSoX5czzBN+8uGiukIH0lUduiy531L33OZ+cf1xZsy77PxOGaZPi?=
 =?us-ascii?Q?MbL7e7GipTFdIOrETyKn8idVy9jwQ47U6RaO4i9OY+baCZ8suYlQL1p0voPg?=
 =?us-ascii?Q?tbdVhauaFUqAxQy1Kwr2EFVALa6d/IliLHyijAxS8l+lo4FjOGUFC0HTlt0A?=
 =?us-ascii?Q?R6QyObWrwBhtdK3JEpLvKGKXjnVwe2C1uHmM9yBG+SZGm9UxPB4xVVQN/XCP?=
 =?us-ascii?Q?W/NZvzw9rcBc0nsVAaXI4OPcR7+OiI/RNkCOeKMlvQiSVPigWQ5h52XEBxou?=
 =?us-ascii?Q?qFJOmV/pjTwrh/+8GEs6zD3m8567oXQq7IEgr0QgIF2Ion6JvGdGHnUTcL84?=
 =?us-ascii?Q?VlqAp0LkhZ79d+EKvYxMuipE5GZlvUgCpKyVy82/Yz3etixd3JigARJyZe64?=
 =?us-ascii?Q?nPmqfEx/GNKa50YSWJBcNvYWtNiuw2Cu+1YHioSRIL6eBXx7bJTM8VwF9CyU?=
 =?us-ascii?Q?4b1Zms1sHNeOxLdOdEKo0bRqdF1yODLVq6qICJKuyLAy07uwMo9rmaPJ06Wp?=
 =?us-ascii?Q?Sf656RLEEPRpKK1fq7KLrad/hh91XnOpNSU/BJK6WpOqukGHjy1ncc5s+QCd?=
 =?us-ascii?Q?QH4MhayQrwWYFBJltiyCt4nTqxYtKjmjgfA6/TcVXU9hT6PE5mla4zxir7P0?=
 =?us-ascii?Q?u5453Wb1WyB47PuSstsqg6YfvgcdMhySkm4KWTREzD8BiQo9tKUppdfMbIDG?=
 =?us-ascii?Q?a1H1HoDtsA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 189eaa62-c40b-478d-3fa6-08da21d9d878
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB4333.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2022 07:54:35.3631
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lVocgmDA3SEF3JohsHvK/URVhPT518CsHL1Bki+wSaJER7KTILA/gRr5voJ5B1Mo
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5952
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE autolearn=no
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Add device managed action to sync pending queue work, otherwise
the queued work may run after the work is destroyed.

Signed-off-by: Li Jun <jun.li@nxp.com>
---
- new patch

 drivers/extcon/extcon-ptn5150.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/drivers/extcon/extcon-ptn5150.c b/drivers/extcon/extcon-ptn5150.c
index 5b9a3cf8df26..2a7874108df8 100644
--- a/drivers/extcon/extcon-ptn5150.c
+++ b/drivers/extcon/extcon-ptn5150.c
@@ -194,6 +194,13 @@ static int ptn5150_init_dev_type(struct ptn5150_info *info)
 	return 0;
 }
 
+static void ptn5150_work_sync_and_put(void *data)
+{
+	struct ptn5150_info *info = data;
+
+	cancel_work_sync(&info->irq_work);
+}
+
 static int ptn5150_i2c_probe(struct i2c_client *i2c)
 {
 	struct device *dev = &i2c->dev;
@@ -284,6 +291,10 @@ static int ptn5150_i2c_probe(struct i2c_client *i2c)
 	if (ret)
 		return -EINVAL;
 
+	ret = devm_add_action_or_reset(dev, ptn5150_work_sync_and_put, info);
+	if (ret)
+		return ret;
+
 	/*
 	 * Update current extcon state if for example OTG connection was there
 	 * before the probe
-- 
2.25.1