From patchwork Sun Feb 16 15:09:31 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dario Binacchi X-Patchwork-Id: 1238764 X-Patchwork-Delegate: lokeshvutla@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=quarantine dis=none) header.from=libero.it Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=libero.it header.i=@libero.it header.a=rsa-sha256 header.s=s2014 header.b=QTBehS4F; dkim-atps=neutral Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 48L9Xw104rz9s29 for ; Mon, 17 Feb 2020 02:10:40 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 59EF68079C; Sun, 16 Feb 2020 16:10:24 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=libero.it Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=libero.it header.i=@libero.it header.b="QTBehS4F"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 2BA7781280; Sun, 16 Feb 2020 16:10:16 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FORGED_SPF_HELO, FREEMAIL_FROM, SPF_HELO_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from libero.it (smtp-35.italiaonline.it [213.209.10.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 515BE8079C for ; Sun, 16 Feb 2020 16:10:12 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=libero.it Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=dariobin@libero.it Received: from localhost.localdomain ([80.116.32.220]) by smtp-35.iol.local with ESMTPA id 3LYRjOBjiMAUp3LYijm1vn; Sun, 16 Feb 2020 16:10:12 +0100 x-libjamoibt: 1601 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=libero.it; s=s2014; t=1581865812; bh=JCxdO37otNF3BMILLVQ5WNtt6i+fkpSbiFP+WzDK2QM=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=QTBehS4FvhjWf86ZuQURx9OguClwBxldDK2I6Vlraz9hSBABGpE2NGdqW5V8uIrDu 1zaFZNd7dBEy+5v1YY0DLRqpEBnxbFLa+kfMFjOLSvAt8f0v66nNXDIPbrHJjJ1nDY oufum6R792Fcn/GFjSZuNyVRsxG1pNT8KnNvZvfRg26mtZaWNqNp+aDHou4Bu8TMFJ /XKL3dKzQISW6cYppFVJMMeUlj7/Vv4Sr5JBvQ2+AO1GkVNeO6px+JvOjhmdh9Tgea rH4PYyBXrQRdS+PqKbJZPnkrmxrJNRETwnPuQRNaRTnt3jipyk/ZeYka7UbE2i65nE qSWINruWNqLrg== X-CNFS-Analysis: v=2.3 cv=B/fHL9lM c=1 sm=1 tr=0 a=gDoL7ROJeSFQT6fuE5q7ug==:117 a=gDoL7ROJeSFQT6fuE5q7ug==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=ng6qe6RrQ1DxvtDK4KcA:9 From: Dario Binacchi To: u-boot@lists.denx.de Cc: Dario Binacchi , Anatolij Gustschin , Lokesh Vutla Subject: [PATCH v2 02/13] video: omap: add missing bitfield masks Date: Sun, 16 Feb 2020 16:09:31 +0100 Message-Id: <20200216150942.3220-3-dariobin@libero.it> X-Mailer: git-send-email 2.24.0 In-Reply-To: <20200216150942.3220-1-dariobin@libero.it> References: <20200216150942.3220-1-dariobin@libero.it> MIME-Version: 1.0 X-CMAE-Envelope: MS4wfOlk+GIFDlqxQWZwlpwZ9ru/i64DcVs1w3RhMXxnYkj9P+UljB79ORgq5OOcWwfWjtCOoH4tY3SHfGQnaWn4G6Eq7BwAtul1w2j5ax6LS2Btc2grT73O og0jRCcr/vHbHXpitLHRhN92ukMbQyzMGcezuX3AhiU7rT0mO+s2nphqNmtFnj8wR6b1ioSuKZQoRL7WRNkICba9qGO9vIRtLWexBbiyFHoHOnZ+SjgrpOTm WvmBTMMWRhzD2u18voKkOSr76Eqr6ftFnifeduYAjjk= X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.30rc1 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.102.1 at phobos.denx.de X-Virus-Status: Clean Add, if missing, the bitfield masks in the setting macros of the LCD controller registers. Signed-off-by: Dario Binacchi - Use GENMASK macro --- Changes in v2: None drivers/video/am335x-fb.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/video/am335x-fb.c b/drivers/video/am335x-fb.c index 7065d57148..4fd3b7f65a 100644 --- a/drivers/video/am335x-fb.c +++ b/drivers/video/am335x-fb.c @@ -26,14 +26,14 @@ #define LCDC_FMAX 200000000 /* LCD Control Register */ -#define LCD_CLK_DIVISOR(x) ((x) << 8) +#define LCD_CLK_DIVISOR(x) (((x) & GENMASK(7, 0)) << 8) #define LCD_RASTER_MODE BIT(0) /* LCD Clock Enable Register */ #define LCD_CORECLKEN BIT(0) #define LCD_LIDDCLKEN BIT(1) #define LCD_DMACLKEN BIT(2) /* LCD DMA Control Register */ -#define LCD_DMA_BURST_SIZE(x) ((x) << 4) +#define LCD_DMA_BURST_SIZE(x) (((x) & GENMASK(2, 0)) << 4) #define LCD_DMA_BURST_1 0x0 #define LCD_DMA_BURST_2 0x1 #define LCD_DMA_BURST_4 0x2 @@ -46,9 +46,9 @@ #define LCD_HORLSB(x) (((((x) >> 4) - 1) & GENMASK(5, 0)) << 4) #define LCD_HORMSB(x) (((((x) >> 4)-1) & 0x40) >> 4) /* LCD Timing_1 Register */ -#define LCD_VBP(x) ((x) << 24) -#define LCD_VFP(x) ((x) << 16) -#define LCD_VSW(x) (((x)-1) << 10) +#define LCD_VBP(x) (((x) & GENMASK(7, 0)) << 24) +#define LCD_VFP(x) (((x) & GENMASK(7, 0)) << 16) +#define LCD_VSW(x) ((((x) - 1) & GENMASK(5, 0)) << 10) #define LCD_VERLSB(x) (((x) - 1) & GENMASK(9, 0)) /* LCD Timing_2 Register */ #define LCD_HSWMSB(x) ((((x) - 1) & GENMASK(9, 6)) << 21)