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 E6C13EB64DD for ; Mon, 26 Jun 2023 20:20:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject: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=jWwuLw/v6FQjYME6BLoWPjGwX41dsQnomP9D2lL4jes=; b=z80M7dE+uGH5HO I2uTeJv+Ivw58oMdjbNjhiugq+6HS2IQYsDKoOLRqcZLTiOMepLj3qUldmkIFeivKYme/XfSYi8tk mzR8EOWnC00lsPE7u11S18HKEQXXbPjaS0zf72LBBCD7trAa2Yl/dj44/AH7/fdoczLn3gH1y3xPG UHveQ4m/u1DhQPHW/0w8fof1zkxSd0CJ87Z3yK/98KVlbc7dWUpyfsm+34nLMCyVDrjNoL1092Svw HRn+TdN6M9VReiRngvSGeNgs0Jx4oUb547RlsCNISpfxazLvWXrikNbYxTUdG2ikU49upXaYCul/k skY+Iw1eASaxLWZgsaVw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qDsgd-00AzeL-04; Mon, 26 Jun 2023 20:19:47 +0000 Received: from mailrelay6-1.pub.mailoutpod2-cph3.one.com ([2a02:2350:5:405::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qDsgY-00AzbH-0b for linux-arm-kernel@lists.infradead.org; Mon, 26 Jun 2023 20:19:46 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ravnborg.org; s=rsa1; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=HH3O/h9U4e4x7ApDSU3+rbilP/iWCVhv/eDCHxh4kVo=; b=T0TBnQT1eRYj5+PK6O9b4JezgAE8HGUuf3EqIznR7yjTP3byGuUGZVB0NpVM6v0NHph2S8l4w4Yig RjUEcYC0k2YHs4/+LJGzSA2RxcAK5JW8p4V70Y1MPcDZ17vHxbFD6pLQ3/ZwuAjcVGXztElBaUE9TL M72+HdDXfZS7JmvQY2LRAG6yOstR07M5g141BR18GFhnJJSODXcRVOI0PwmqI+dKo8a1zS297XHkjr Kh3SCqUL5Ig4ssxpIsSly36JxhtM1UXvTE9jv73Iv1dZ6f3DKFYxHLYRK7gKlZ/2ACkVapwX4IuEWy tpnhgNnNqYU3edETCo1325cYSH+mcsQ== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=ravnborg.org; s=ed1; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=HH3O/h9U4e4x7ApDSU3+rbilP/iWCVhv/eDCHxh4kVo=; b=9y4tez6lwQh+SQWeTUZSHthxVuGzolmPlBCCnJAJwH1wb6OSjx61Brut1eWWY1b8SxG6Tx6go5INa 2mLwHLnDg== X-HalOne-ID: b28f4f00-145e-11ee-8bce-6f01c1d0a443 Received: from ravnborg.org (2-105-2-98-cable.dk.customer.tdc.net [2.105.2.98]) by mailrelay6 (Halon) with ESMTPSA id b28f4f00-145e-11ee-8bce-6f01c1d0a443; Mon, 26 Jun 2023 20:19:08 +0000 (UTC) Date: Mon, 26 Jun 2023 22:19:06 +0200 From: Sam Ravnborg To: Manikandan Muralidharan Subject: Re: [PATCH 3/9] drm: atmel-hlcdc: add LCD controller layer definition for SAM9X7 Message-ID: <20230626201906.GA11422@ravnborg.org> References: <20230613070426.467389-1-manikandan.m@microchip.com> <20230613070426.467389-4-manikandan.m@microchip.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230613070426.467389-4-manikandan.m@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230626_131942_429380_36F5E92A X-CRM114-Status: GOOD ( 16.61 ) 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: , Cc: Nayabbasha.Sayed@microchip.com, devicetree@vger.kernel.org, alexandre.belloni@bootlin.com, daniel@ffwll.ch, bbrezillon@kernel.org, linux-kernel@vger.kernel.org, Balamanikandan.Gunasundar@microchip.com, lee@kernel.org, conor+dt@kernel.org, dri-devel@lists.freedesktop.org, Varshini.Rajendran@microchip.com, Dharma.B@microchip.com, robh+dt@kernel.org, Durai.ManickamKR@microchip.com, krzysztof.kozlowski+dt@linaro.org, Hari.PrasathGE@microchip.com, Balakrishnan.S@microchip.com, airlied@gmail.com, claudiu.beznea@microchip.com, linux-arm-kernel@lists.infradead.org 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 Manikandan, On Tue, Jun 13, 2023 at 12:34:20PM +0530, Manikandan Muralidharan wrote: > Add the LCD controller layer definition and descriptor structure for SAM9X7 > for the following layers, > - Base Layer > - Overlay1 Layer > - Overlay2 Layer > - High End Overlay > > Signed-off-by: Manikandan Muralidharan > --- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 96 ++++++++++++++++++++ > 1 file changed, 96 insertions(+) > > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > index fa0f9a93d50d..d7ad828e9e8c 100644 > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > @@ -462,6 +462,98 @@ static const struct atmel_hlcdc_dc_desc atmel_hlcdc_dc_sam9x60 = { > .layers = atmel_hlcdc_sam9x60_layers, > }; > > +static const struct atmel_hlcdc_layer_desc atmel_xlcdc_sam9x7_layers[] = { > + { > + .name = "base", > + .formats = &atmel_hlcdc_plane_rgb_formats, > + .regs_offset = 0x60, > + .id = 0, > + .type = ATMEL_HLCDC_BASE_LAYER, > + .cfgs_offset = 0x1c, > + .layout = { > + .xstride = { 2 }, > + .default_color = 3, > + .general_config = 4, > + .disc_pos = 5, > + .disc_size = 6, > + }, > + .clut_offset = 0x700, > + }, > + { > + .name = "overlay1", > + .formats = &atmel_hlcdc_plane_rgb_formats, > + .regs_offset = 0x160, > + .id = 1, > + .type = ATMEL_HLCDC_OVERLAY_LAYER, > + .cfgs_offset = 0x1c, > + .layout = { > + .pos = 2, > + .size = 3, > + .xstride = { 4 }, > + .pstride = { 5 }, > + .default_color = 6, > + .chroma_key = 7, > + .chroma_key_mask = 8, > + .general_config = 9, > + }, > + .clut_offset = 0xb00, > + }, > + { > + .name = "overlay2", > + .formats = &atmel_hlcdc_plane_rgb_formats, > + .regs_offset = 0x260, > + .id = 2, > + .type = ATMEL_HLCDC_OVERLAY_LAYER, > + .cfgs_offset = 0x1c, > + .layout = { > + .pos = 2, > + .size = 3, > + .xstride = { 4 }, > + .pstride = { 5 }, > + .default_color = 6, > + .chroma_key = 7, > + .chroma_key_mask = 8, > + .general_config = 9, > + }, > + .clut_offset = 0xf00, > + }, > + { > + .name = "high-end-overlay", > + .formats = &atmel_hlcdc_plane_rgb_and_yuv_formats, > + .regs_offset = 0x360, > + .id = 3, > + .type = ATMEL_HLCDC_OVERLAY_LAYER, > + .cfgs_offset = 0x30, > + .layout = { > + .pos = 2, > + .size = 3, > + .memsize = 4, > + .xstride = { 5, 7 }, > + .pstride = { 6, 8 }, > + .default_color = 9, > + .chroma_key = 10, > + .chroma_key_mask = 11, > + .general_config = 12, > + .csc = 16, > + .scaler_config = 23, > + }, > + .clut_offset = 0x1300, > + }, > +}; > + > +static const struct atmel_hlcdc_dc_desc atmel_xlcdc_dc_sam9x7 = { > + .min_width = 0, > + .min_height = 0, > + .max_width = 2048, > + .max_height = 2048, > + .max_spw = 0xff, > + .max_vpw = 0xff, > + .max_hpw = 0x3ff, > + .fixed_clksrc = true, > + .nlayers = ARRAY_SIZE(atmel_xlcdc_sam9x7_layers), > + .layers = atmel_xlcdc_sam9x7_layers, > +}; As already suggested by someone else, add is_xlcdc to struct atmel_hlcdc_dc_desc, so the check for the compatible can be dropped. It would be better to put it here. Sam _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel