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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E7420C02180 for ; Wed, 15 Jan 2025 17:55:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From :Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=is/Az9xGToTsuvQhxSyLFLcXvYqhkADF/5ojbpYHtBU=; b=1uaUdzmO11dlp6nqGyFUyggYiY 6nWKMG5GmSgLoDB5kH7K0l20txDpbpAcXhyURcUxGG+KYueoR3n0iH7NKzatsspH/NxRltWzcLnCm mhuf6/5cD7A3agZbAjRh81E21NOTNwLQsiKyTiHSnTykopr174qKfk2kMg0GZfn8QOXoZFN7GOKmc 2sB4vrlULK/Y2gj8p8MH0d1IEuRDeSYk2hl3U+0KP6vDcZs52TngmeNwzbOi8fywZw+6cjOPaNF2x PH0OpnShIE69UKMunV2OHPEmFSMavh6pt1aRaOH4aoAxtCcj0Q7K7ybLVihmVMNPPxmzM8WCLXG0k IzY/NqCg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tY7bx-0000000CghU-0VBi; Wed, 15 Jan 2025 17:55:25 +0000 Received: from 009.lax.mailroute.net ([199.89.1.12]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tY7aL-0000000Cg0E-0I1K; Wed, 15 Jan 2025 17:53:46 +0000 Received: from localhost (localhost [127.0.0.1]) by 009.lax.mailroute.net (Postfix) with ESMTP id 4YYDBc2k4gzlgMVW; Wed, 15 Jan 2025 17:53:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=acm.org; h= content-transfer-encoding:content-type:content-type:in-reply-to :from:from:content-language:references:subject:subject :user-agent:mime-version:date:date:message-id:received:received; s=mr01; t=1736963619; x=1739555620; bh=is/Az9xGToTsuvQhxSyLFLcX vYqhkADF/5ojbpYHtBU=; b=Ng1loLEuc/xiqIXDqAxKT0RhMLuNdkB/9Ztk1mYg n0Izfs2wFJeOI8t6HeumaGvHR/ZMQ57s34/kmp5CKve9jiZp7LxOGsqpgxBeAhDc gXuzQ1VptPRYXKa7lq52Du9YpK6NchNqwhDtNWkSiUqJb0L2jPYugDgXrHXczv1z KDrO4ji9MW/EE86h8Ak0ao3rSqvuiQ1XOTqcTPrXCeGWzqCELR0he/gxxjZLvVft AHK/62cFZEUlRWvl/XjzCki6DA5xIwSl5vsxvV+Zfv7DENIOUaBrpfGqAMIaqZ87 9vKHr2cJ9RSbkMewHkPgVzOYKtVxHNQTE1Ha5y9hyLALPw== X-Virus-Scanned: by MailRoute Received: from 009.lax.mailroute.net ([127.0.0.1]) by localhost (009.lax [127.0.0.1]) (mroute_mailscanner, port 10029) with LMTP id 0lmg9n-RFN6D; Wed, 15 Jan 2025 17:53:39 +0000 (UTC) Received: from [100.66.154.22] (unknown [104.135.204.82]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: bvanassche@acm.org) by 009.lax.mailroute.net (Postfix) with ESMTPSA id 4YYDBR19J1zlgT1M; Wed, 15 Jan 2025 17:53:34 +0000 (UTC) Message-ID: Date: Wed, 15 Jan 2025 09:53:33 -0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] ufs: Use str_enable_disable-like helpers To: Krzysztof Kozlowski , Alim Akhtar , Avri Altman , "James E.J. Bottomley" , "Martin K. Petersen" , Peter Wang , Stanley Jhu , Matthias Brugger , AngeloGioacchino Del Regno , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org References: <20250114200716.969457-1-krzysztof.kozlowski@linaro.org> Content-Language: en-US From: Bart Van Assche In-Reply-To: <20250114200716.969457-1-krzysztof.kozlowski@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250115_095345_133091_B4D6A298 X-CRM114-Status: UNSURE ( 9.33 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On 1/14/25 12:07 PM, Krzysztof Kozlowski wrote: > 2. Is slightly shorter thus also easier to read. Does this change really make code easier to read? It forces readers of the code to look up a function definition. Isn't there a general preference in the Linux kernel to inline function definitions if the function body is shorter than or close to the length of the function name? I'm referring to functions like this one: static inline const char *str_up_down(bool v) { return v ? "up" : "down"; } Bart.