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=-5.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,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 DDB90C47089 for ; Thu, 27 May 2021 08:44:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BAA1A613CA for ; Thu, 27 May 2021 08:44:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235560AbhE0Ipj (ORCPT ); Thu, 27 May 2021 04:45:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53812 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235568AbhE0Ipc (ORCPT ); Thu, 27 May 2021 04:45:32 -0400 Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6BE80C06138C for ; Thu, 27 May 2021 01:43:57 -0700 (PDT) Received: by mail-wr1-x42d.google.com with SMTP id n2so3914766wrm.0 for ; Thu, 27 May 2021 01:43:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=3L68bacZr0PwfRAm8hFSUVoyJv0NBxuRAhL667AuTWc=; b=nDtehq73M+4wZnrQ176gTUypOQxMAGzZCelnnQEMdA4Ue1zkPgmV5Ug2vLEydPtU3e eMf3mTHDAaV15LfakVzjNi5cY7RDxfHOuS7nNga/XElt4NMKG2bJ5F/WwXmG7SL+tvvt uXYNA4oVYp3nx/7qm8bQwKt+ohvE/IBiF37Im/hO0RmY40gMVSVWxlGtE6t7SCy86xIi KL7vPmbghjcDkIOUfXxM+zZwOvyu/usKu0iC68SKUEJgMBGQ74j9pRRg2CQ/cA0k64D/ YzqcoCffyAhgqFo175DLeM5qu16yhWz8e4RR28stArmEyWWJdZ+uvQbRwRtscaEHvSN4 mWPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=3L68bacZr0PwfRAm8hFSUVoyJv0NBxuRAhL667AuTWc=; b=TknPhfq8DDbruJrxHq8w90VYmnGxDtA/X04WPAZmO/1/qPMe+bBa6UVIwkiYpG/V2c jhnKtfVNAllREjN4/S3L11mfGKhM74YDHHzWcJ0AbUM9SFZrbI5pPfRHA54EchzUsV8I De0ncwW6E19x2FVc+0UuYevktK0U3ZRgfvHCyia7FG6t836f2S6o+qEHeDgMu19BsHfN /V05wcou5ATJjGWu9kQaA97sXftCkLGnWLpbun7VBYO0Sgeh+hCCKCx/ZqrKg+wuq04h HYEoBuH1hJRjvcGEMxPPodDoJP0DkldUAbyYdIGUYKNKfXSVUDhx/kA6OjUZFRB1oj8l LKdg== X-Gm-Message-State: AOAM531QuHrSMqkQq/wKmQpoLZzh5brN2xRVZhb2wphsjYYuTwJu9vtS EPBxKiOP9DYIOnvTSUC2/7jQiw== X-Google-Smtp-Source: ABdhPJwpUHtR6b1jBYiaOEHNxKmLgkQf8QQbbg9t709yLkdkhRxReXsSbCUv4edxfYoBMyHsZT2+7Q== X-Received: by 2002:adf:decf:: with SMTP id i15mr2149088wrn.300.1622105035953; Thu, 27 May 2021 01:43:55 -0700 (PDT) Received: from dell ([91.110.221.223]) by smtp.gmail.com with ESMTPSA id h67sm2160856wrh.44.2021.05.27.01.43.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 27 May 2021 01:43:55 -0700 (PDT) Date: Thu, 27 May 2021 09:43:53 +0100 From: Lee Jones To: Min Li Cc: "sameo@linux.intel.com" , "grant.likely@linaro.org" , "robh+dt@kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" Subject: Re: [PATCH mfd v1] mfd: Add Renesas Synchronization Management Unit (SMU) support Message-ID: <20210527084353.GJ543307@dell> References: <1619466246-11198-1-git-send-email-min.li.xe@renesas.com> <20210519080807.GH805368@dell> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, 26 May 2021, Min Li wrote: > > > > > + ret = rsmu_dev_init(rsmu); > > > + mutex_unlock(&rsmu->lock); > > > + > > > + if (ret == 0) > > > + dev_info(rsmu->dev, "Probe %s successful\n", > > > + rsmu_name(rsmu->type)); > > > > No thank you. Please remove this. > > > Hi Lee > > What is the rule for adding informational log in the driver probe function? The rule generally is 'don't'. Especially if it doesn't provide any actual information. Strings are expensive with regards to binary size. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog