From patchwork Tue Dec 5 22:56:38 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Randy Dunlap X-Patchwork-Id: 13480852 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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 97DD1C4167B for ; Tue, 5 Dec 2023 22:56:43 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C4C1C10E550; Tue, 5 Dec 2023 22:56:41 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by gabe.freedesktop.org (Postfix) with ESMTPS id 30F8810E550 for ; Tue, 5 Dec 2023 22:56:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:Cc:To:From:Sender:Reply-To:Content-Type: Content-ID:Content-Description:In-Reply-To:References; bh=MdnCEOu4uestWpLQY50JohpBreItAAs8W53VxcosDAU=; b=ymfUUOVOkplMqLDWizo3PRMhzT R9qmoWAu0gaQAm+eDR80jbVU2HAW4AM0Z+gTuKm+EZ+IwEseBOgPwJXR3wupm6KwmosWjuK8GEKBK fb/hEH68kxHJZseFJB3aDKXnVaJoXIJXnTe0rOBC5VfUD4mSs1URhm7y/XdykSvUDlGIuho7EhtSa TbMcL64jMz0RN+h7GlJUNXH/assVxEM/HnTCpE/8CXCDt2qkVVgeYKMe2fnwySd0bEdnTIbZDxLsh ai51ofOYvEytzc8BZoUC+bfN1fi+XTbtQ4w0iLZtMwVdrWrdvY1I0I4YtVggqaOSeBSwPc3DE6Dnc ijYg64tg==; Received: from [50.53.46.231] (helo=bombadil.infradead.org) by bombadil.infradead.org with esmtpsa (Exim 4.96 #2 (Red Hat Linux)) id 1rAeLG-008YxJ-38; Tue, 05 Dec 2023 22:56:39 +0000 From: Randy Dunlap To: dri-devel@lists.freedesktop.org Subject: [PATCH] backlight: ili922x: fix W=1 kernel-doc warnings Date: Tue, 5 Dec 2023 14:56:38 -0800 Message-ID: <20231205225638.32563-1-rdunlap@infradead.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Daniel Thompson , Lee Jones , Jingoo Han , Helge Deller , Randy Dunlap , linux-fbdev@vger.kernel.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Fix kernel-doc warnings found when using "W=1". ili922x.c:85: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst ili922x.c:85: warning: missing initial short description on line: * START_BYTE(id, rs, rw) ili922x.c:91: warning: contents before sections ili922x.c:118: warning: expecting prototype for CHECK_FREQ_REG(spi_device s, spi_transfer x)(). Prototype was for CHECK_FREQ_REG() instead Signed-off-by: Randy Dunlap Cc: Lee Jones Cc: Daniel Thompson Cc: Jingoo Han Cc: Helge Deller Cc: linux-fbdev@vger.kernel.org --- drivers/video/backlight/ili922x.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff -- a/drivers/video/backlight/ili922x.c b/drivers/video/backlight/ili922x.c --- a/drivers/video/backlight/ili922x.c +++ b/drivers/video/backlight/ili922x.c @@ -82,13 +82,12 @@ #define START_RW_READ 1 /** - * START_BYTE(id, rs, rw) - * - * Set the start byte according to the required operation. + * START_BYTE() - Set the start byte according to the required operation. * The start byte is defined as: * ---------------------------------- * | 0 | 1 | 1 | 1 | 0 | ID | RS | RW | * ---------------------------------- + * * @id: display's id as set by the manufacturer * @rs: operation type bit, one of: * - START_RS_INDEX set the index register @@ -101,14 +100,14 @@ (0x70 | (((id) & 0x01) << 2) | (((rs) & 0x01) << 1) | ((rw) & 0x01)) /** - * CHECK_FREQ_REG(spi_device s, spi_transfer x) - Check the frequency + * CHECK_FREQ_REG() - Check the frequency * for the SPI transfer. According to the datasheet, the controller * accept higher frequency for the GRAM transfer, but it requires * lower frequency when the registers are read/written. * The macro sets the frequency in the spi_transfer structure if * the frequency exceeds the maximum value. * @s: pointer to an SPI device - * @x: pointer to the read/write buffer pair + * @x: pointer to the &spi_transfer read/write buffer pair */ #define CHECK_FREQ_REG(s, x) \ do { \