Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp282213rwd; Tue, 16 May 2023 00:37:09 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4NcyZBRhNgRKUXnViHr5O4U0iaHgoOQ3uWswoG49R/SI6PF5nmZMYBnEtPjsKwvuPcEp8z X-Received: by 2002:a05:6a20:3d8e:b0:106:76e8:940a with SMTP id s14-20020a056a203d8e00b0010676e8940amr6096158pzi.61.1684222629094; Tue, 16 May 2023 00:37:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684222629; cv=none; d=google.com; s=arc-20160816; b=TZnIFJGW3ZLGSC4B5d/AjCAMmtThUJO2ZpqLjPWV+R/0ZxqpFqwr3JhAAA36l1eG4V bfQRyZ5BvvPdq1FS7+ODGnLTIyJICqUrbxDLoIby/+OTZDvsGn81HFUukUQbH1GWyU/Y kT0o29VJA1bdk0AvxMtOCzqSit/2GuLpML7peCMLWx8Ax8ELvF6w0wOagIiNrO8Mq3KX A7uuywR2cxIktzy9Dq4O2RXmRZVexG/QgoC0yQN+nr1p1Io3g4/5kjTP+z/vQllv37zy w5WQmbj6efASCQ16umDq9USya87ehJ2wwhMKom0GdN7WKueAxdwtjpF5a3U+Ni0asJJ1 b5mA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:references:cc:to:content-language:subject:reply-to :from:user-agent:mime-version:date:message-id:dkim-signature; bh=mriKUD+ePrXG2DRWD5gePTJ4sodjlKWYwwduKOxs8DI=; b=UAOR+iJhF62gJmmkmCTfmvAIz05I9DxIljP6XXyZgD0qKHVwOQBl59slvaygxXUx6p za0rAFmAl0VMwmzSKUJAdHz/kUFTz5sclisSrV0OUYxD+tr9skTzhzndSd2e71o87f5d uWgdBHX9lPiYHayjGpazvsxN/DYCw7TCscuG0AqkS+5CIQWJEc+SbDw6ZpZOECPo45Jn PnrRUBl1wX7uDnwCE9dbdOWXMYyWWvmo2kpQ2L2LjreHXsO+4a78L/O/sZsEfiIh7ktq FBNbNT9RkTfAZ1N06pxT6bgmhaJABwfY79tsRzPLWwwFLBoN81N/tUlCEL4keDCXD5UX CGqg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=ST7RVOqL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w25-20020a637b19000000b005130f15c5b1si11803420pgc.363.2023.05.16.00.36.56; Tue, 16 May 2023 00:37:09 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=ST7RVOqL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231197AbjEPHYf (ORCPT + 99 others); Tue, 16 May 2023 03:24:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57988 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231173AbjEPHYd (ORCPT ); Tue, 16 May 2023 03:24:33 -0400 Received: from mail-wr1-x42a.google.com (mail-wr1-x42a.google.com [IPv6:2a00:1450:4864:20::42a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2E0341FDA for ; Tue, 16 May 2023 00:24:28 -0700 (PDT) Received: by mail-wr1-x42a.google.com with SMTP id ffacd0b85a97d-3090408e09bso2628681f8f.2 for ; Tue, 16 May 2023 00:24:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1684221866; x=1686813866; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=mriKUD+ePrXG2DRWD5gePTJ4sodjlKWYwwduKOxs8DI=; b=ST7RVOqLOXdhN4AiR9ZC4VSzODuLCFiQUdBe6NgMcrufpcWwPo6JVRYDK5b+vltIZP 1kfmvNZ4dqGp0NINctE6TF4Uho6L+j7NuBrkGaybeHGliwi9c4ARwxdSfYI067feVfJ6 PLs1SyMVNcrw2cVCuXKWzPQJ4tucHmRMrBQi0yNye/HqqcEl2JBTY79hrHoIbNA/VO/m 8yx/rM9qno6IF7169ccBU5OoNTegeJqbnQXGOfD1IlaIcx+/FgsFkq020uoXm/H+r9WM DSdn34eAIwgarpCEaQs/evNpITPAlBQrwwxCkWgMqRaMIGDGbEV4vXE+kEoNifFU31+T WGsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684221866; x=1686813866; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=mriKUD+ePrXG2DRWD5gePTJ4sodjlKWYwwduKOxs8DI=; b=PfJiyqdA+sqqsTrGWxwPbg+ngF9LqOKSp1k9j2MMr799bDs7tW8k6BVbbeAdYhV0SI 620K3shR02GbyGqGugUyLhyz3hoNuf31DjxsvU8lm/KVPWM2iJ/QiP1MSTja4hCwQpsJ PdmkF6o90OZy3oFOyQonnphocn8i5RvdyRFV0dsuYbwabEShLzZxz4ij383xW5dMCo+K YVvOr+kUkXf1Xen62T7cd+6e1dIPsRAK1TA93ywv17r6R7+pq6+MRCk8p3WvtWcQOQ/t /QkY7UNBl4P4WyqC9pJKdvoyx9FXuO3PA/rxuQ5ml/UyzLvgMzBDXaYFLh9vL8k/KK7K S4ZQ== X-Gm-Message-State: AC+VfDxObsEgV1gg6oB0GvFgs0mF/8Z88VJIz1aWb8LhDtjiPqf9C27Z DY4nJ+MKRUFNPl4+Eceddh0o7Q== X-Received: by 2002:adf:f245:0:b0:306:2d28:d556 with SMTP id b5-20020adff245000000b003062d28d556mr22320151wrp.34.1684221866463; Tue, 16 May 2023 00:24:26 -0700 (PDT) Received: from ?IPV6:2a01:e0a:982:cbb0:86be:97a:a043:77a8? ([2a01:e0a:982:cbb0:86be:97a:a043:77a8]) by smtp.gmail.com with ESMTPSA id r7-20020adff107000000b003012030a0c6sm1548409wro.18.2023.05.16.00.24.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 16 May 2023 00:24:25 -0700 (PDT) Message-ID: Date: Tue, 16 May 2023 09:24:24 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 From: Neil Armstrong Reply-To: neil.armstrong@linaro.org Subject: Re: [PATCH v6 4/8] drm/bridge: mhdp8546: Set input_bus_flags from atomic_check Content-Language: en-US To: Aradhya Bhatia , Tomi Valkeinen , Jyri Sarha , David Airlie , Daniel Vetter , Laurent Pinchart , Andrzej Hajda , Robert Foss , Jonas Karlman , Jernej Skrabec , Rahul T R , Swapnil Jakhade , Boris Brezillon , Francesco Dolcini Cc: DRI Development List , Linux Kernel List , Nishanth Menon , Vignesh Raghavendra , Devarsh Thakkar , Jayesh Choudhary References: <20230509093036.3303-1-a-bhatia1@ti.com> <20230509093036.3303-5-a-bhatia1@ti.com> Organization: Linaro Developer Services In-Reply-To: <20230509093036.3303-5-a-bhatia1@ti.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-5.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/05/2023 11:30, Aradhya Bhatia wrote: > From: Nikhil Devshatwar > > input_bus_flags are specified in drm_bridge_timings (legacy) as well > as drm_bridge_state->input_bus_cfg.flags > > The flags from the timings will be deprecated. Bridges are supposed > to validate and set the bridge state flags from atomic_check. > > Signed-off-by: Nikhil Devshatwar > [a-bhatia1: replace timings in cdns_mhdp_platform_info by input_bus_flags] > Signed-off-by: Aradhya Bhatia > --- > > Notes: > > changes from v5: > * removed the wrongly addded return statement in tfp410 driver. > * replaced the timings field in cdns_mhdp_platform_info by > input_bus_flags field, in order to get rid of bridge->timings > altogether. > > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c | 11 ++++++++--- > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h | 2 +- > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c | 9 ++++----- > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.h | 2 +- > 4 files changed, 14 insertions(+), 10 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c > index 623e4235c94f..a677b1267525 100644 > --- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c > +++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c > @@ -2189,6 +2189,13 @@ static int cdns_mhdp_atomic_check(struct drm_bridge *bridge, > return -EINVAL; > } > > + /* > + * There might be flags negotiation supported in future. > + * Set the bus flags in atomic_check statically for now. > + */ > + if (mhdp->info) > + bridge_state->input_bus_cfg.flags = *mhdp->info->input_bus_flags; > + > mutex_unlock(&mhdp->link_mutex); > return 0; > } > @@ -2554,8 +2561,6 @@ static int cdns_mhdp_probe(struct platform_device *pdev) > mhdp->bridge.ops = DRM_BRIDGE_OP_DETECT | DRM_BRIDGE_OP_EDID | > DRM_BRIDGE_OP_HPD; > mhdp->bridge.type = DRM_MODE_CONNECTOR_DisplayPort; > - if (mhdp->info) > - mhdp->bridge.timings = mhdp->info->timings; Won't this cause a breakage because at this point in time bridge.timings->input_bus_flags seems to be still used by tidss right ? Neil > > ret = phy_init(mhdp->phy); > if (ret) { > @@ -2642,7 +2647,7 @@ static const struct of_device_id mhdp_ids[] = { > #ifdef CONFIG_DRM_CDNS_MHDP8546_J721E > { .compatible = "ti,j721e-mhdp8546", > .data = &(const struct cdns_mhdp_platform_info) { > - .timings = &mhdp_ti_j721e_bridge_timings, > + .input_bus_flags = &mhdp_ti_j721e_bridge_input_bus_flags, > .ops = &mhdp_ti_j721e_ops, > }, > }, > diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h > index bedddd510d17..bad2fc0c7306 100644 > --- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h > +++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h > @@ -336,7 +336,7 @@ struct cdns_mhdp_bridge_state { > }; > > struct cdns_mhdp_platform_info { > - const struct drm_bridge_timings *timings; > + const u32 *input_bus_flags; > const struct mhdp_platform_ops *ops; > }; > > diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c > index dfe1b59514f7..12d04be4e242 100644 > --- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c > +++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.c > @@ -71,8 +71,7 @@ const struct mhdp_platform_ops mhdp_ti_j721e_ops = { > .disable = cdns_mhdp_j721e_disable, > }; > > -const struct drm_bridge_timings mhdp_ti_j721e_bridge_timings = { > - .input_bus_flags = DRM_BUS_FLAG_PIXDATA_SAMPLE_NEGEDGE | > - DRM_BUS_FLAG_SYNC_SAMPLE_NEGEDGE | > - DRM_BUS_FLAG_DE_HIGH, > -}; > +const u32 > +mhdp_ti_j721e_bridge_input_bus_flags = DRM_BUS_FLAG_PIXDATA_SAMPLE_NEGEDGE | > + DRM_BUS_FLAG_SYNC_SAMPLE_NEGEDGE | > + DRM_BUS_FLAG_DE_HIGH; > diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.h b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.h > index 97d20d115a24..5ddca07a4255 100644 > --- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.h > +++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-j721e.h > @@ -14,6 +14,6 @@ > struct mhdp_platform_ops; > > extern const struct mhdp_platform_ops mhdp_ti_j721e_ops; > -extern const struct drm_bridge_timings mhdp_ti_j721e_bridge_timings; > +extern const u32 mhdp_ti_j721e_bridge_input_bus_flags; > > #endif /* !CDNS_MHDP8546_J721E_H */