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=-14.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 676E5C433ED for ; Wed, 12 May 2021 16:43:46 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B235461040 for ; Wed, 12 May 2021 16:43:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B235461040 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=OjqlPhV6rIDyB8M4H+1/CpNtXFhNrRRW36xI8GtdVdc=; b=BErqZ63IB25kD90xqupCJdSE7 ACboYzJmIbUxD1C2M1/d5yrbV3PGd5spS2G8jkl204JSKNIEYUD0dlqet67siyAVwO+L+KRZOEZ7b iCdG8F9yM3RpeJO2U9JpE+dvR7o9ZzKiCFIejcVMULt4y8hiwg9erDygth1ceF6h2LdXGL0FFlrrS MeiCUV048N2wyR7y2VOVEMlQ5SIZ0owjAjfLKdQlTfgkDTv1KYmbITeMMZ64oLfT+X37x+mD2rcnk uN16mvbxo0NCvxSq6uwx9oYN3t7MSJzd5fI29ZJS+H9C1ytcVL+qB6PlJ2/QpqWREwTPP41XiWhbL e1kU6M88A==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lgrwB-003N6o-3c; Wed, 12 May 2021 16:42:19 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lgrw6-003N6G-Mw for linux-arm-kernel@desiato.infradead.org; Wed, 12 May 2021 16:42:14 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=c0VLlOA4O95JRL4lKkh9nBVXdg9khLB5Tb87tWSnVRw=; b=qNu4DkVCZqT8AKR66AbgDNGEfR h5Ex01Xv1wWcK5bWS3T7yerBUCs1U1tQpflpoaOdkbypsApm4/F5yQ14voPsNdJkvW7sfcxISkgJX W4PJh9PhGIjWPDebDSS8PBoNxBEUzpWUwdibNZLlz87sSBKxty0q20b+YHbfkzb2YqptqD9XEr6DB 8bpmKB8oqQ4axXmVPnuB9fP4w2N/FyT4RxeqnHbWZIPtXKmaQHo96ECMK/yohePLnUheFk+i1CmpF iEJrgERvdXb25GuvtXasPortjIaLLHXU24/2ouanVTaZV4c5zCgNWnKMQQK1H99BM1/mlMlJArIAm 9nPNJ9bA==; Received: from mail-pf1-x42a.google.com ([2607:f8b0:4864:20::42a]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lgrw4-00AbuE-30 for linux-arm-kernel@lists.infradead.org; Wed, 12 May 2021 16:42:13 +0000 Received: by mail-pf1-x42a.google.com with SMTP id b21so11840602pft.10 for ; Wed, 12 May 2021 09:42:10 -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:in-reply-to; bh=c0VLlOA4O95JRL4lKkh9nBVXdg9khLB5Tb87tWSnVRw=; b=GLyllHtwAmx2LbR5f2ncGAhINUHQKpdLq0Va8A1jrEEH4iKxyi8yPxjQgDESkJIxI8 sgoLTQVM7dgUoN+y2VlM6E/Bw5o4DjbV/YhVFPA0LnMq0wbCkCTecSfv6o4de75oC4YH Za+cMwPJonToYd5+eHdZv9MjkDrusuoKNZRgO+4txsra0GzgH0ZKK+r+2SQ4qhaKEsEy BZz4C1Wy5j+1iGlQWVlY368VZYQ8G82YQHoSKQMXhYNhcbuA08OH9xoUZ49vl/hWDUnc OlnJB/RukALauzfpxqTJekUSL0z1GfxqrbmZAZ8WATDNa0pCbh2J9jeF3PqUqvF2ZlQ/ KcEA== 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:in-reply-to; bh=c0VLlOA4O95JRL4lKkh9nBVXdg9khLB5Tb87tWSnVRw=; b=ckxBtuZXf89AUr5Fg2jiLkXqC5gSNy9Ymmf102bfhaKZHDUy5y7aAli9EyHHhx+Hud C7HqBbKudP0rWjyjWV+UuI0YFr3Wt/t9p3645OyU/1e7Pi0CyMZIyZDNHrJOhQoYx4sX n5B9/s/KTEFfKO8sxj1FhEcpkJ+dKhXLoOv93vGp6e6sc1T2bN/Cp1qpStTQM1vwEfy3 osmmKCtY4t5V7QuFsSlGfVg0Puu1OMSDjOpc/O3p2jJbHo61Fks4BUc86jsqb8rDSAnh 030MYqo+DqGwvRtPThykTON09UPX/EOGNO4It74VFTeIMKo8tZ7nEhG9ZV9LU+LDfogl kJTg== X-Gm-Message-State: AOAM532w8QR64cpVFGTnCRSlca3BY4GGDu+TzRmAVKJ2okS6V5sY7aRe MWbb+anfy8/MkOztXj4HqEGTPzT5Z5z0zQ== X-Google-Smtp-Source: ABdhPJzgnbgnzOYlNJGD1LlShrIF+jXPWbxIzI+AR97Zitg5NpGAK4/drZctKAFXCgiOqJMxGrmBPA== X-Received: by 2002:a63:6387:: with SMTP id x129mr36767474pgb.58.1620837730005; Wed, 12 May 2021 09:42:10 -0700 (PDT) Received: from xps15 (S0106889e681aac74.cg.shawcable.net. [68.147.0.187]) by smtp.gmail.com with ESMTPSA id n25sm299936pfo.92.2021.05.12.09.42.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 May 2021 09:42:08 -0700 (PDT) Date: Wed, 12 May 2021 10:42:06 -0600 From: Mathieu Poirier To: Zhen Lei Cc: Ohad Ben-Cohen , Bjorn Andersson , Matthias Brugger , linux-remoteproc , linux-mediatek , linux-arm-kernel Subject: Re: [PATCH 1/1] remoteproc/mediatek: Remove redundant error printing in scp_probe() Message-ID: <20210512164206.GA186911@xps15> References: <20210511120426.5834-1-thunder.leizhen@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210511120426.5834-1-thunder.leizhen@huawei.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210512_094212_166859_13331414 X-CRM114-Status: GOOD ( 22.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Lei, On Tue, May 11, 2021 at 08:04:26PM +0800, Zhen Lei wrote: > When devm_ioremap_resource() fails, a clear enough error message will be > printed by its subfunction __devm_ioremap_resource(). The error > information contains the device name, failure cause, and possibly resource > information. > > Therefore, remove the error printing here to simplify code and reduce the > binary size. > > Reported-by: Hulk Robot > Signed-off-by: Zhen Lei > --- > drivers/remoteproc/mtk_scp.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/remoteproc/mtk_scp.c b/drivers/remoteproc/mtk_scp.c > index 9679cc26895e578..edb7f7842cb12ad 100644 > --- a/drivers/remoteproc/mtk_scp.c > +++ b/drivers/remoteproc/mtk_scp.c > @@ -749,7 +749,6 @@ static int scp_probe(struct platform_device *pdev) > res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "sram"); > scp->sram_base = devm_ioremap_resource(dev, res); > if (IS_ERR((__force void *)scp->sram_base)) { > - dev_err(dev, "Failed to parse and map sram memory\n"); This kind of construct can be found several times in the remoteproc subsystem and is also ubiquitous throughout the kernel. Moveover removing the error notification does not simplify the code and does little to reduce the binary size. As such I do not think this patch is warranted and will not move forward with it. Thanks, Mathieu > ret = PTR_ERR((__force void *)scp->sram_base); > goto free_rproc; > } > -- > 2.26.0.106.g9fadedd > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel