Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp490907imu; Tue, 20 Nov 2018 02:25:12 -0800 (PST) X-Google-Smtp-Source: AFSGD/ULpHPXynHm1HERosIp3d2GWHDsqB5w5mLHvTFnP9sOczqpZkGwavaxOUzJDKeTufNxhadb X-Received: by 2002:a63:fb46:: with SMTP id w6mr1362855pgj.321.1542709512495; Tue, 20 Nov 2018 02:25:12 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542709512; cv=none; d=google.com; s=arc-20160816; b=XZGuNH01Im2siDdNcwpYv9XJnulNpKPqn+VCbpoMULV9GnP/nKbRHMllrGY9bxtln2 y1Lnjcj4EGDNzY/qpQVdj3VPeyOX8L5+W3LhgoHgHq8NRNrQvmc/CHF+QvEeElOXkQZv wZ5F29936yXG4hZYZx7QoxNAHKKYNNgugvAr1bObYRWIE9cG40DXIvtMp3LjP9KktBpY VL6uJvaXUYM4aDVb9Sdeqfv7Z+XESeZbJ5SU86N+u55zoL0qQ27I1UcASSVhxrgaeGRI 3TJqn89HYUpF2QbdfAMEjgKnf1RYWgpGP9NBsR8GrphJfAFEmhRVEu6J5DZ43+gdgJAM GaAA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:date:cc:to:from:subject:message-id; bh=t+FK1NTZ+f42y0eqFGXFl0t/zCV9GpA6jWtEqElNbH4=; b=BKeZHD0yib0O5F1xAclkoQKE9pV2p/2FVxO/iLwS2z7Dm9BHtn9TiB8/kqATW2AUZO S33gAEn6sHjsCk3GJHtZ+6beumnK07TIs4dqYnhxIn9nkvdYRfxBILtq3z1/r+lcVaSO Hd3Uhfk5pLfAdrgOyjhwNQY/jgW8M3T25Hqsyzsnr9P/k9xvWHCaE5sOQ2xeyaeOk/SK CwwF8ZW0IgSnnMpoqeXb9tInOGtQyYMrAUGIFMF0yHTJ0WdfqTYUkXyAP+dPruel20J5 tvorJMc5PXIDYmQ3HsQu9PaEGOjx6ieXDM4OQg1WsI792Xas4S/mnyfFJcg4gH+hthIw N5ZQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u22si6803747pgh.286.2018.11.20.02.24.57; Tue, 20 Nov 2018 02:25:12 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727346AbeKTUvx (ORCPT + 99 others); Tue, 20 Nov 2018 15:51:53 -0500 Received: from mailgw02.mediatek.com ([210.61.82.184]:9752 "EHLO mailgw02.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1726250AbeKTUvx (ORCPT ); Tue, 20 Nov 2018 15:51:53 -0500 X-UUID: f617e93033b3491781c07a1ff9cd1b68-20181120 X-UUID: f617e93033b3491781c07a1ff9cd1b68-20181120 Received: from mtkexhb01.mediatek.inc [(172.21.101.102)] by mailgw02.mediatek.com (envelope-from ) (mhqrelay.mediatek.com ESMTP with TLS) with ESMTP id 1875717253; Tue, 20 Nov 2018 18:23:25 +0800 Received: from mtkcas08.mediatek.inc (172.21.101.126) by mtkmbs02n1.mediatek.inc (172.21.101.77) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Tue, 20 Nov 2018 18:23:23 +0800 Received: from [172.21.77.4] (172.21.77.4) by mtkcas08.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1395.4 via Frontend Transport; Tue, 20 Nov 2018 18:23:23 +0800 Message-ID: <1542709403.13494.0.camel@mtksdaap41> Subject: Re: [PATCH v5 05/12] drm: mediatek: Omit warning on probe defers From: CK Hu To: Matthias Brugger CC: , , , , , , , , , , , , , , , , , , , Matthias Brugger Date: Tue, 20 Nov 2018 18:23:23 +0800 In-Reply-To: <9a445746-b07d-73e1-68c9-10eb9d5bf771@gmail.com> References: <20181116125449.23581-1-matthias.bgg@kernel.org> <20181116125449.23581-6-matthias.bgg@kernel.org> <1542605939.32082.12.camel@mtksdaap41> <0a02edf9-78da-dde9-4b69-38c4719b711b@gmail.com> <1542686700.9073.15.camel@mtksdaap41> <9a445746-b07d-73e1-68c9-10eb9d5bf771@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-MTK: N Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Matthias: On Tue, 2018-11-20 at 11:19 +0100, Matthias Brugger wrote: > > On 20/11/2018 05:05, CK Hu wrote: > > Hi, Matthias: > > > > On Mon, 2018-11-19 at 10:26 +0100, Matthias Brugger wrote: > >> > >> On 19/11/2018 06:38, CK Hu wrote: > >>> Hi, Matthias: > >>> > >>> On Fri, 2018-11-16 at 13:54 +0100, matthias.bgg@kernel.org wrote: > >>>> From: Matthias Brugger > >>>> > >>>> It can happen that the mmsys clock drivers aren't probed before the > >>>> platform driver gets invoked. The platform driver used to print a warning > >>>> that the driver failed to get the clocks. Omit this error on > >>>> the defered probe path. > >>> > >>> This patch looks good to me, but you have not modified the sub driver in > >>> HDMI path. We could let HDMI path print the warning and someone send > >>> another patch later, or you modify for HDMI path in this patch. > >> > >> Sure, I'll add this in v6. After inspecting the code, I think we will need to > >> also check for not initialized clocks in mtk_mdp_comp_init, as the driver for > >> now does not even check if the clocks are present. What do you think? > > > > Yes, we do really need to consider mdp driver because mmsys clock > > include mdp clock. You remind me that mmsys control 4 major function: > > drm routing, drm clock, mdp routing, and mdp clock. Your design let the > > mmsys device as drm device (control drm routing) and create a sub device > > as clock device (control drm clock, mdp clock). If one day mdp device > > (may need control drm routing) need to control the register of mdp > > routing, would mdp device be a sub device? Or we need not to consider > > this because it need not to access mmsys register now? > > > > I think we should for now concentrate to fix the clock probing issue. If in the > future we will need to access drm routing from the mdp device, we can have a > look into this. > > Sounds good? Sounds good to me. Regards, CK > Regards, > Matthias