Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp5005307yba; Wed, 10 Apr 2019 09:18:55 -0700 (PDT) X-Google-Smtp-Source: APXvYqzm4xppr6qmmdZ/AY0VlYUY3H9dYT6vjyd/ezgjc2da8FvIWuyoKo7zf3tqT+BiQlem1uI/ X-Received: by 2002:a62:12c8:: with SMTP id 69mr44546401pfs.184.1554913134899; Wed, 10 Apr 2019 09:18:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1554913134; cv=none; d=google.com; s=arc-20160816; b=m1/0loKe73ycBqQ+wIhmu55pTsASUAMp4Y6kLWgH6Zb8NgVificrzDvJgeovNb3Xux TBIlcl432U6q363cNVSgujnfrdD4eMDQaT6BwyvwMbff917rImV7ROJztuYvtjvucjFM ProUtG2/cgmCAGUQ74mgzEdvE6x9Na8JqZp3F/vA0iNIQNCaO7ZOq7tkFE6yiKxIVDjw m2CsdvQCvdo9kno8U9jumqEiHJFz/I4XCA+5uHxww/RPYTqKuOOG4B4qWDkfQ6Wmh3p+ 4g/XEvKrByoAIa2UE8DhEw92IAftQwqTwjbgS14fNLzVcNCgO8zLYksEfuV8kJaYi3i9 vMiQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=PoRP+LOtTSnagnjX2I5zTkt2ACY+HyZwRMrB06M2b1w=; b=KUFFXpsBCA3HqAJxwCmiqi+ftHEerV7roWSoTOmpMKIcL+gzdIzobLw8GrNbm4qHiV M3a2UmrujLEE0J/UAWU537QgXDm6yobZceSfpYtpitLcZ2EuHGiwe3gtwATpmwDaXwmg ZGe9r3YMiEkZP7EvenTDqYVv4h/j4+VJ9di6x6mZFfkzPl0vYzqRUHD7qhy+05jlS1vY Q0xnXuVQffeQD3aUH5Ve9xmZg0bc9QS+oDUOAuxSQqdTClRTe82AVJnFs4PzfYHA7Dbk OtcbGEQcABPrEOFefc/lT8GsYpuoSp63SKjMEtDrZ1vRFymRn/fCWQukFDAzEpPuaYKA B5Bg== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f11si16537963plo.169.2019.04.10.09.18.39; Wed, 10 Apr 2019 09:18:54 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730347AbfDJNE6 (ORCPT + 99 others); Wed, 10 Apr 2019 09:04:58 -0400 Received: from ns.iliad.fr ([212.27.33.1]:44546 "EHLO ns.iliad.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727855AbfDJNE6 (ORCPT ); Wed, 10 Apr 2019 09:04:58 -0400 Received: from ns.iliad.fr (localhost [127.0.0.1]) by ns.iliad.fr (Postfix) with ESMTP id B941D21873; Wed, 10 Apr 2019 15:04:56 +0200 (CEST) Received: from [192.168.108.8] (freebox.vlq16.iliad.fr [213.36.7.13]) by ns.iliad.fr (Postfix) with ESMTP id 9CEA42094A; Wed, 10 Apr 2019 15:04:56 +0200 (CEST) Subject: Re: [PATCH 23/23] watchdog: tangox_wdt: Convert to use device managed functions and other improvements To: Guenter Roeck , Wim Van Sebroeck Cc: LKML , Mans Rullgard , Dmitry Torokhov References: <1554830641-9247-1-git-send-email-linux@roeck-us.net> <1554830641-9247-24-git-send-email-linux@roeck-us.net> From: Marc Gonzalez Message-ID: <6ebee4d6-c650-c322-a966-4269d54da9be@free.fr> Date: Wed, 10 Apr 2019 15:04:56 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: <1554830641-9247-24-git-send-email-linux@roeck-us.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV using ClamSMTP ; ns.iliad.fr ; Wed Apr 10 15:04:56 2019 +0200 (CEST) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/04/2019 19:24, Guenter Roeck wrote: > Use device managed functions to simplify error handling, reduce > source code size, improve readability, and reduce the likelyhood of bugs. > Other improvements as listed below. > > The conversion was done automatically with coccinelle using the > following semantic patches. The semantic patches and the scripts > used to generate this commit log are available at > https://github.com/groeck/coccinelle-patches > > - Drop assignments to otherwise unused variables > - Drop unnecessary braces around conditional return statements > - Drop empty remove function > - Use devm_add_action_or_reset() for calls to clk_disable_unprepare > - Replace stop on remove with call to watchdog_stop_on_unregister() > - Use devm_watchdog_register_driver() to register watchdog device No devm_clk_prepare() in mainline? :-( https://lore.kernel.org/patchwork/patch/755487/ Regards.