Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 21 Jun 2022 11:35:26 +0530
From:   Rahul T R <r-ravikumar@ti.com>
To:     Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
CC:     <dri-devel@lists.freedesktop.org>, <andrzej.hajda@intel.com>,
        <narmstrong@baylibre.com>, <robert.foss@linaro.org>,
        <jonas@kwiboo.se>, <jernej.skrabec@gmail.com>, <airlied@linux.ie>,
        <daniel@ffwll.ch>, <p.zabel@pengutronix.de>,
        <laurent.pinchart@ideasonboard.com>,
        <linux-kernel@vger.kernel.org>, <jpawar@cadence.com>,
        <sjakhade@cadence.com>, <mparab@cadence.com>, <a-bhatia1@ti.com>
Subject: Re: [PATCH v2] drm/bridge: cdns-dsi: Add support for J721E wrapper
Message-ID: <20220621060525.kvm5wojt6kmfqhqv@uda0490373>
References: <20220619140158.30881-1-r-ravikumar@ti.com>
 <042c77d5-7db1-fa09-4be4-74dbfa85b5e2@ideasonboard.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <042c77d5-7db1-fa09-4be4-74dbfa85b5e2@ideasonboard.com>
User-Agent: NeoMutt/20171215
Precedence: bulk

On 13:02-20220620, Tomi Valkeinen wrote:
> Hi,
> 
> On 19/06/2022 17:01, Rahul T R wrote:
> > Add support for wrapper settings for DSI bridge on
> > j721e. Also set the DPI input to DPI0
> > 
> > Signed-off-by: Rahul T R <r-ravikumar@ti.com>
> > ---
> 
> Nack... This wouldn't work with some other SoC using CDNS DSI.
> 
> See cdns-mhdp8546 for an example of a bit more generic wrapper support.
> 
>  Tomi

Thanks Tomi,

I have sent a respin with making changes similar to cdns-mhdp8546
please review

Regards
Rahul T R