From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.7 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 06A88C432C2 for ; Tue, 24 Sep 2019 14:10:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D297A21655 for ; Tue, 24 Sep 2019 14:10:33 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20150623.gappssmtp.com header.i=@baylibre-com.20150623.gappssmtp.com header.b="zBxpjFaP" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2436882AbfIXOKa (ORCPT ); Tue, 24 Sep 2019 10:10:30 -0400 Received: from mail-wr1-f44.google.com ([209.85.221.44]:40029 "EHLO mail-wr1-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729970AbfIXOKa (ORCPT ); Tue, 24 Sep 2019 10:10:30 -0400 Received: by mail-wr1-f44.google.com with SMTP id l3so2130755wru.7 for ; Tue, 24 Sep 2019 07:10:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:in-reply-to:references:date:message-id :mime-version; bh=CD8NXjWHXI1PHdT+Yt5q2iCIoHrQv0Ncx4K7iwz1VKY=; b=zBxpjFaP7l2EinUCWp0CpW2Sg1KTXXAkWldWE1wejALSHl3jp0sFDuHfH9Gi2HC2Au kzQ8eNjBepwuBE+WbnhNaJeaA2YgNuzXB5kwhQzzLXinKPvQCT716XwpcJ0OQwEW4Q1k mYBZVNfAVOI9FS3MUBcZFqmEz8BMf5scqsMrQSYS7a0Jda+62JkKZCi45L2ODbr/7ITr EoZCg4hKA2dNdppOt/WLSZcVso9BHBuKo+dAoQSrK79NYYuSijPjb9/ZAnfpmak9bhlW Khimacswau6lOgHXL5Nurct25X6UOV0+d8wHpX6mo9I/A7DTGUsuFceEWn1DJa/iOWon b9dw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=CD8NXjWHXI1PHdT+Yt5q2iCIoHrQv0Ncx4K7iwz1VKY=; b=cOCldxzKcOzgemhL4c8R5vvUQYlBu0ip1Kk9hsyeHM8l1l1SIf6NvTHN2doFbGFPAf t7Ijk/UmxMQr7XqhZ6znVnca/D0JptyRBK5Dk0TUImO95dC6KAVRrHsI/Zy98CHMGEtw 24A//hW3EC/CyBf4rB17cmHgpVOt2cZmjuI1YE9FvXV9ry9i0Xvg5r2FHUzTdAZ7VNIi cnOSv/ApkXlm4XNfjFl9lyU+a7nb2Af9TmBczb/QLCUkkkHv1lAXBAmkiL9aVxwhP9ek /aVzVDvXBraRyE5hRZBowFz2kJggjXA0QqJapxPXfYRVRGUkUw2Nz3R5OkoBIw/QnZJd 5m4Q== X-Gm-Message-State: APjAAAX/SBpqHP8djKWn71qJYJRqPGnIEDJ8r1QIqvLoOheweEcRflEk 6guzYVhGcBjFoors2qeArXj7DQ== X-Google-Smtp-Source: APXvYqyxcFFWxlfEByHp+UI2Ozbsm+7ZlCpzeuoY03TS06zxKMIrzCq38xzQB0qK8eF54jJ/spcLAg== X-Received: by 2002:adf:e7ca:: with SMTP id e10mr2442564wrn.234.1569334227683; Tue, 24 Sep 2019 07:10:27 -0700 (PDT) Received: from localhost (uluru.liltaz.com. [163.172.81.188]) by smtp.gmail.com with ESMTPSA id f83sm61195wmf.43.2019.09.24.07.10.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 24 Sep 2019 07:10:26 -0700 (PDT) From: Jerome Brunet To: Ankur Tyagi , Michael Turquette , Stephen Boyd Cc: "linux-clk\@vger.kernel.org" , "linux-kernel\@vger.kernel.org" , Heiko Stuebner , Tero Kristo , Andrew Lunn , Florian Fainelli , Heiner Kallweit , "David S. Miller" , "netdev\@vger.kernel.org" , "linux-amlogic\@lists.infradead.org" , "linux-arm-msm\@vger.kernel.org" , "linux-rockchip\@lists.infradead.org" , "linux-omap\@vger.kernel.org" Subject: RE: [PATCH 2/3] clk: let init callback return an error code In-Reply-To: References: <20190924123954.31561-1-jbrunet@baylibre.com> <20190924123954.31561-3-jbrunet@baylibre.com> Date: Tue, 24 Sep 2019 16:10:25 +0200 Message-ID: <1jv9thlr8u.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 Content-Type: text/plain Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue 24 Sep 2019 at 13:38, Ankur Tyagi wrote: > Hi, > > I am no expert here but just looked at the patch and found few > discrepancy that I have mentioned inline. > [...] > > Aren't all functions returning 0 always? > Yes, on purpose. This patch is an API conversion to let the init() callback of the clock ops return an error code or 0. The patch is not meant to change anything in the prior behavior of the clock drivers which is why every exit path return 0 with this change. IOW, yes there are all returning 0 for now, but it will eventually change. >> * >> * @debug_init:Set up type-specific debugfs entries for this clock. This >> *is called once, after the debugfs directory entry for this >> @@ -243,7 +247,7 @@ struct clk_ops { >> struct clk_duty *duty); >> int(*set_duty_cycle)(struct clk_hw *hw, >> struct clk_duty *duty); >> -void(*init)(struct clk_hw *hw); >> +int(*init)(struct clk_hw *hw); >> void(*debug_init)(struct clk_hw *hw, struct dentry *dentry); >> }; >> >> -- >> 2.21.0 > > ________________________________ > This email is confidential and may contain information subject to legal privilege. If you are not the intended recipient please advise us of our error by return e-mail then delete this email and any attached files. You may not copy, disclose or use the contents in any way. The views expressed in this email may not be those of Gallagher Group Ltd or subsidiary companies thereof. > ________________________________