Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp608398pxb; Fri, 29 Oct 2021 16:37:54 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyoRxY84EM7o/pkQEPpqRBN+xOYKaT2a2RishodpUujE9c0bjsN550WRTJF7wz67t4eyDGe X-Received: by 2002:a05:6402:270f:: with SMTP id y15mr19082903edd.256.1635550674618; Fri, 29 Oct 2021 16:37:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1635550674; cv=none; d=google.com; s=arc-20160816; b=FCR7lLqPiY7u2G4BrbpBOv4gHrGOSI5FvvWKZopKUSUlwuoCV1cbt8gNSkbUv8qK40 0r8tWbCJXXU87twVCplGTQ9DCpuTnMs3Juyhl4YVL8I8jeVyMTZSrfLwK8U+EQbxOnfU k5VOdVG4EK357pbVxsesYXa1oEtr6niHMWx7dBqdLqv0qEsh41WDPjrTSEwnlSN1H7Fj /1L4FfnZVaGLpWt3nSLPLyOxmgwbFYTQbN0iCV63mpCa0BEcXsbwEJB0a7dLfhK1Hbhl b72+xUMmUDoYtMeXJX4AN+gEnacreYn98bqJTPg75T5sm5BNIO3Q2A/0Lh6LeDmgMqb5 U0nA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:user-agent:from :references:in-reply-to:mime-version:dkim-signature; bh=PpuLCz4ga4Gy0Gh5NWyC/cRzLKCgj8drSbwewDSyv/8=; b=pXA0ZT6AED12ftO1ICDZ1yFaIm5Iy496wvw54Byi6VA40l9rMejJDqpC0PiNS8ccwT AZJfMIlFVm6WI85jTeG90I03ix0/vbYIdEOZbn69gKEgkQwzr0PeEyLAGdt3KcW7YOPN Z7Ypjg0BMIuZt8c/Ee/EqldckvCSQUA0KHISli5XspnTq2AUn6O1bqGwuXYwWDjfhy8f SvUxb3Cqiy90K+dzs4Va5eLWCgmIk6P4R1HXi0ZCzgSyje1pqBcmk/PCkWw9BRsFGA1z yXbxfs+uJ0jrnXGK6IVsx1P5x/CzGtbsHT37usS6c8tEDTLuAEzdWQVxaYf7Y4wx7P2i HeZQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Oypysc3N; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id fy37si8417463ejc.764.2021.10.29.16.37.16; Fri, 29 Oct 2021 16:37:54 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Oypysc3N; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231592AbhJ2Xht (ORCPT + 99 others); Fri, 29 Oct 2021 19:37:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56260 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231546AbhJ2Xhm (ORCPT ); Fri, 29 Oct 2021 19:37:42 -0400 Received: from mail-oi1-x22c.google.com (mail-oi1-x22c.google.com [IPv6:2607:f8b0:4864:20::22c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 64EE2C061766 for ; Fri, 29 Oct 2021 16:35:13 -0700 (PDT) Received: by mail-oi1-x22c.google.com with SMTP id m11so10773881oif.13 for ; Fri, 29 Oct 2021 16:35:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:in-reply-to:references:from:user-agent:date:message-id :subject:to:cc; bh=PpuLCz4ga4Gy0Gh5NWyC/cRzLKCgj8drSbwewDSyv/8=; b=Oypysc3NkHamf8CiPca5foDaX4rg23TW2UfyCEtKHki7XkSRFky2YWN++WJHWmmJfM mkbIXUlJiHkGkFVw0c/Wd75fdP21uSCZwExRYIbzXB2Rigp33corJIq+pUtKGGsxtcri X/+yJqOEoqN2mgG9ALqWp7+NFJ1ZXJhqcrTLc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:in-reply-to:references:from :user-agent:date:message-id:subject:to:cc; bh=PpuLCz4ga4Gy0Gh5NWyC/cRzLKCgj8drSbwewDSyv/8=; b=IdrAb3QyaN/vGOjHyDvVWxdkaRlW0wOY8NimI7QA1ISjVoUfkKAMumpnpf69BDRY2M 4KAckpGDWQxKa3/WqVpgSF5LuFaeA3ULU0JrDADmXHjyNulTU3XZC8aFSceeX+mLp4H9 CU93ph+mqjuGRbFIWaRpBTf4Zjh6OuLyf0Sdixwc/v2kwF/6aoBbDrmKAS/hbcAJdsiP ssWSTKRIiQ1CQFPG3QPpBB/4YfWNjhbWJYSXmqK1McgLTufsMCoxsdhXWcspXjeEan4i m3T6/ISI4pYTxwwu9g0bmPXKhlWFwXAMt0b1MzRIeTdrRErqLZ5/JIp/CI57V3PtzF/K YBrQ== X-Gm-Message-State: AOAM5324KO8Yl7OQvH6mFzopllSHJNx59C1LN/cWNLwqFc4sDgwe+owk ur/XfyQqZsQVWwGA7f6BL+Q99rl8E2mq4I9m/bNuQw== X-Received: by 2002:a05:6808:2307:: with SMTP id bn7mr2111471oib.32.1635550512672; Fri, 29 Oct 2021 16:35:12 -0700 (PDT) Received: from 753933720722 named unknown by gmailapi.google.com with HTTPREST; Fri, 29 Oct 2021 18:35:12 -0500 MIME-Version: 1.0 In-Reply-To: References: <20211029152647.v3.1.Ie17e51ad3eb91d72826ce651ca2786534a360210@changeid> <20211029152647.v3.2.If23c83a786fc4d318a1986f43803f22b4b1d82cd@changeid> From: Stephen Boyd User-Agent: alot/0.9.1 Date: Fri, 29 Oct 2021 18:35:12 -0500 Message-ID: Subject: Re: [PATCH v3 2/4] arm64: dts: sc7180: Specify "data-lanes" for DSI host output To: Philip Chen Cc: LKML , dianders@chromium.org, Andy Gross , Bjorn Andersson , Rob Herring , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Philip Chen (2021-10-29 16:24:58) > Hi Stephen, > > On Fri, Oct 29, 2021 at 4:16 PM Stephen Boyd wrote: > > > > Quoting Philip Chen (2021-10-29 15:27:41) > > > MSM DSI host driver actually parses "data-lanes" in DT and compare > > > it with the number of DSI lanes the bridge driver sets for > > > mipi_dsi_device. So we need to always specify "data-lanes" for the > > > DSI host output. As of now, "data-lanes" is added to ti-sn65dsi86 dts > > > fragment, but missing in parade-ps8640 dts fragment, which requires > > > a fixup. > > > > I don't see data-lanes required in the schema, and > > dsi_host_parse_lane_data() seems happy to continue without it. I do see > > that num_data_lanes isn't set though. Does this patch fix it? > The problem I see is from dsi_host_attach(). > If there is no "data-lanes" in DT, num_data_lanes would be 0. > Then dsi_host_attach() would return -EINVAL. Ok, got it. > > > > > ----8<---- > > diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c > > b/drivers/gpu/drm/msm/dsi/dsi_host.c > > index e269df285136..f6fba07220e5 100644 > > --- a/drivers/gpu/drm/msm/dsi/dsi_host.c > > +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c > > @@ -1696,6 +1696,7 @@ static int dsi_host_parse_lane_data(struct > > msm_dsi_host *msm_host, > > if (!prop) { > > DRM_DEV_DEBUG(dev, > > "failed to find data lane mapping, using default\n"); > > + msm_host->num_data_lanes = 4; > > return 0; > > } > > I haven't tried. > But I think it can fix the problem I described above. > Would you like to send it as a separate patch? > Or I can do it. Sure feel free to send it as another patch. Or fix the schema to make data-lanes required. I think fixing the driver is probably better so that we don't have to set data-lanes when it's the default 4 lanes. At least I think 4 lanes is the default.