Received: by 2002:ac0:950c:0:0:0:0:0 with SMTP id f12csp3611653imc; Thu, 14 Mar 2019 00:48:51 -0700 (PDT) X-Google-Smtp-Source: APXvYqyQ7AMRpAEu8ggQUoKSNiyRHlqKOAZPNFfpeJjkNJKgRTtdL3TbhYdBzDRMY8ODfsarUNUB X-Received: by 2002:a63:fe0d:: with SMTP id p13mr42431353pgh.366.1552549731370; Thu, 14 Mar 2019 00:48:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1552549731; cv=none; d=google.com; s=arc-20160816; b=f2wTpDju9uhZ/vcPvCWzbq/MfztN5LdJQW9V4ARlEBmYnuac10lYHhCiBHNtg+vyTr dmlClzyN8nnjgdGUATgC2O6n8zVCGrX4xIDvrqNFLxKL6VNxlRxPyvPawTKQ+1/BKDNZ hjPCXJM9exVwbsUSPzWE0AL49/pHu28wXE9ffRJI94HOZeBDwpfGCeu9at57mw/C4Hn/ SrjndDXtrG2IVt1X1ISxZFUHzIc1qnyAo3HWfqVfjBV7WktJ7nkHmkk4oGGyS7xqhCoD Dzn5H2FKGQrazznuuZpVaQrnDqnKFgPqP3nWCvvf7JSkTBf6K6ZD2d2ZL8TEwJpJ4Utf F4Zw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:dlp-filter:cms-type :content-transfer-encoding:date:message-id:in-reply-to:cc:to:from :reply-to:subject:mime-version:dkim-signature:dkim-filter; bh=xNnYwL6CfNzGQuXbhKlugEnLnTYr0q/qVj+CwffWNp4=; b=D8cufkS3LFQaxelsenfk4JWsYa3uzunY0O0i3+A0QZeYf9oWuD2Tb6H5aY4d92DOiY Cm5apYMaMb88a54sCZHiuEiBHhhF2KKfHzcu4hp3rDAdhNDC/6OYoP6M/mG/yn1571Bo mZDCqJVaT8e8b9z0r8QTapELeU29uDgtHEHsAlxWrtuTvzZ9ApYc4QqYp0oJpogiFpQU k7pCsrNYy8cajDQSujJ5pw5oVVk9srUIQRQ0RXV0huVn4GaIL2n+lo9VXczP5KpAuJqZ RxDI4yL9tS1Q8a3fLQMW0L7xhDUE5qw1ur3bRqk2F2wU1OuXAbqDv7oQB9zj15Y7cIq/ GVuA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=U5KgQpFl; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h9si14115602plk.373.2019.03.14.00.48.36; Thu, 14 Mar 2019 00:48:51 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=U5KgQpFl; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727233AbfCNHrn (ORCPT + 99 others); Thu, 14 Mar 2019 03:47:43 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:13470 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727078AbfCNHrm (ORCPT ); Thu, 14 Mar 2019 03:47:42 -0400 Received: from epcas1p3.samsung.com (unknown [182.195.41.47]) by mailout3.samsung.com (KnoxPortal) with ESMTP id 20190314074739epoutp038867c2299507851b95e43de26d841e87~Lw7ByzaiQ0953309533epoutp03r for ; Thu, 14 Mar 2019 07:47:39 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout3.samsung.com 20190314074739epoutp038867c2299507851b95e43de26d841e87~Lw7ByzaiQ0953309533epoutp03r DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1552549659; bh=xNnYwL6CfNzGQuXbhKlugEnLnTYr0q/qVj+CwffWNp4=; h=Subject:Reply-To:From:To:CC:In-Reply-To:Date:References:From; b=U5KgQpFlK9V8Sz1ZUxdjFZNrS8rbej+tuX0rN7VYudNwTM9mhIPoZhZ0uxmo7M8ed oTUZvEIjA5vifFIpM35A6Q9+nZim/gUeRzblYBzAHB+1+Ws0rY5KvJNd8SSCwI4zia TCa8PFvQPMn3BQ1KSlzEKyc1sNIUvZ43EzmKWRqY= Received: from epsmges1p3.samsung.com (unknown [182.195.40.152]) by epcas1p4.samsung.com (KnoxPortal) with ESMTP id 20190314074735epcas1p4137a9e5523723e9f0918a435c781f21d~Lw6_xohwg2414324143epcas1p4A; Thu, 14 Mar 2019 07:47:35 +0000 (GMT) X-AuditID: b6c32a37-f19ff7000000102f-15-5c8a07163aa0 Received: from epcas1p3.samsung.com ( [182.195.41.47]) by epsmges1p3.samsung.com (Symantec Messaging Gateway) with SMTP id E6.4F.04143.6170A8C5; Thu, 14 Mar 2019 16:47:34 +0900 (KST) Mime-Version: 1.0 Subject: RE: [RESEND PATCH] PM / devfreq: Fix static checker warning in try_then_request_governor Reply-To: myungjoo.ham@samsung.com From: MyungJoo Ham To: Enric Balletbo i Serra , "linux-kernel@vger.kernel.org" CC: "kernel@collabora.com" , Dan Carpenter , Chanwoo Choi , Kyungmin Park , "linux-pm@vger.kernel.org" X-Priority: 3 X-Content-Kind-Code: NORMAL In-Reply-To: <20190313122253.24355-1-enric.balletbo@collabora.com> X-Drm-Type: N,general X-Msg-Generator: Mail X-Msg-Type: PERSONAL X-Reply-Demand: N Message-ID: <20190314074734epcms1p8c7c23ea44032c17581fd1bf252bea5fa@epcms1p8> Date: Thu, 14 Mar 2019 16:47:34 +0900 X-CMS-MailID: 20190314074734epcms1p8c7c23ea44032c17581fd1bf252bea5fa Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" CMS-TYPE: 101P X-Brightmail-Tracker: H4sIAAAAAAAAA02Se0hTYRjG/Xa2naM1OS7LD7utQwkaczuu6TE1ujPIQqhQQloHPW3ibuxs lUW17D5aKAkTW9kNBSc4ZupIabakiFK6oWgXpBolmTW0IEht29mo/348vO/zvs/7fRgiPiNM x6oMFsZsoHWEMInf/TBTJl2C2svlI7+k1MjMZwH1dc7Jp9rfBADVOXRJQA3WTqLUq3suITXt GAAbUZXv3R2gCgXH+KrLd9uAatq7ooS/r7pQy9CVjFnCGCqMlVUGTRGxY7d6i1qZKyelZD6V R0gMtJ4pIrYWl0i3V+nCGxCSQ7TOGpZKaJYlZBsKzUarhZFojayliGBMlTpTvimbpfWs1aDJ rjDq15NyeY4yXHigWjtwNYiYhtEjl6ffAhtwC+0gEYP4Ojg/PcazgyRMjPsAHH3Yi9oBhonw FDjrWxSpWYRXQPfEC16ExTgBbaP3EU6XwQvj3SDCQlwKuzzv+BFOxY/CHx8no4zgEwCemirl Zolg4/kgn+OlsKe1K9qbiG+GrvrfCKcvhqPub2icvz9qBhynwrPvB2M1KXD8dy+I+zQ2zMc8 j8Fh5200kgXiZwB0uBtiRTL47NqTaGARvhPaGxqjYfj4GtjvORsz3QpfjvfxuKVXwp5vLiRy BwTPhB33ZJycDKd+XhLEs/iuf+JxnAEDvXdiOy+DT503Y5Yq2GKP3/YigK8bnyN1YGXTv/M2 /Tet6d+0GwBpA0sYE6vXMCxpUvz/ol4Q/YFZeT7gGSoOABwDxEKRDblYLhbQh9gafQBADCFS RXuzw5Kokq45ypiNarNVx7ABoAyHrkfSF1cYw//ZYFGTyhyFQkGtI3OVJEmkiTo2bikX4xra wlQzjIkxx/t4WGK6DdD99J5g8xTqkTpP3GwtOOdyHCxGvX9SglfEZRk+SaffmNY9vrx3+GTo T0JIumZTx+rQ4we1M3a/fCRhgSJ50+mcvIPz0sKJL2vL2uePFwnTNJOncm+hLbxMb2efY/jD 7OF9oQFixq96NbtqrMCdnwT3a7v2KHaVzunrtmX5FQSf1dJkFmJm6b9d1l48lwMAAA== DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20190313122310epcas5p1cc4a6a8570c90ce76743b90a7e5c96c1 References: <20190313122253.24355-1-enric.balletbo@collabora.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >The patch 23c7b54ca1cd: "PM / devfreq: Fix devfreq_add_device() when >drivers are built as modules." leads to the following static checker >warning: > > drivers/devfreq/devfreq.c:1043 governor_store() > warn: 'governor' can also be NULL > >The reason is that the try_then_request_governor() function returns both >error pointers and NULL. It should just return error pointers, so fix >this by returning a ERR_PTR to the error intead of returning NULL. > >Fixes: 23c7b54ca1cd ("PM / devfreq: Fix devfreq_add_device() when drivers are built as modules.") >Reported-by: Dan Carpenter >Signed-off-by: Enric Balletbo i Serra >Reviewed-by: Chanwoo Choi >--- >Hi, > >This is a resend of [1] as seems that got lost at some point and I just >noticed that was never merged. > >Thanks, > Enric Acked-by: MyungJoo Ham Thanks! CHeers, MyungJoo