From: Kevin Tang <kevin3.tang@gmail.com> To: Maxime Ripard <maxime@cerno.tech> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Sean Paul <sean@poorly.run>, David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Orson Zhai <orsonzhai@gmail.com>, Chunyan Zhang <zhang.lyra@gmail.com>, "Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>, ML dri-devel <dri-devel@lists.freedesktop.org>, devicetree@vger.kernel.org Subject: Re: [PATCH v4 6/6] drm/sprd: add Unisoc's drm mipi dsi&dphy driver Date: Thu, 15 Apr 2021 08:19:44 +0800 [thread overview] Message-ID: <CAFPSGXbJwNd47UAwVu4c1t5K7j4sgMtTLtDZ15woHZbRTghVfA@mail.gmail.com> (raw) In-Reply-To: <20210407104854.fgn67pvau7dchcm2@gilmour> Maxime Ripard <maxime@cerno.tech> 于2021年4月7日周三 下午6:48写道: > > On Wed, Mar 31, 2021 at 09:47:12AM +0800, Kevin Tang wrote: > > > > diff --git a/drivers/gpu/drm/sprd/Makefile > > > b/drivers/gpu/drm/sprd/Makefile > > > > index 6c25bfa99..d49f4977b 100644 > > > > --- a/drivers/gpu/drm/sprd/Makefile > > > > +++ b/drivers/gpu/drm/sprd/Makefile > > > > @@ -1,5 +1,8 @@ > > > > # SPDX-License-Identifier: GPL-2.0 > > > > > > > > obj-y := sprd_drm.o \ > > > > - sprd_dpu.o > > > > - > > > > + sprd_dpu.o \ > > > > + sprd_dsi.o \ > > > > + dw_dsi_ctrl.o \ > > > > + dw_dsi_ctrl_ppi.o \ > > > > > > So it's a designware IP? There's a driver for it already that seems > > > fairly similar: > > > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c > > > > > Our dw dsi controller is not a standard synopsys ip, we have updated a lot > > on the basic ip version, > > the entire control register is different, i have cc to drm/bridge reviewers > > and maintainers. > > You should make it more obvious then in a comment or in the name of the > driver. If it's fairly different from the original IP from Synopsys, > maybe you should just drop the reference to the name? Thks, i will be fix it. > > Maxime
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Tang <kevin3.tang@gmail.com> To: Maxime Ripard <maxime@cerno.tech> Cc: Mark Rutland <mark.rutland@arm.com>, devicetree@vger.kernel.org, David Airlie <airlied@linux.ie>, Chunyan Zhang <zhang.lyra@gmail.com>, "Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>, ML dri-devel <dri-devel@lists.freedesktop.org>, Rob Herring <robh+dt@kernel.org>, Orson Zhai <orsonzhai@gmail.com>, Sean Paul <sean@poorly.run> Subject: Re: [PATCH v4 6/6] drm/sprd: add Unisoc's drm mipi dsi&dphy driver Date: Thu, 15 Apr 2021 08:19:44 +0800 [thread overview] Message-ID: <CAFPSGXbJwNd47UAwVu4c1t5K7j4sgMtTLtDZ15woHZbRTghVfA@mail.gmail.com> (raw) In-Reply-To: <20210407104854.fgn67pvau7dchcm2@gilmour> Maxime Ripard <maxime@cerno.tech> 于2021年4月7日周三 下午6:48写道: > > On Wed, Mar 31, 2021 at 09:47:12AM +0800, Kevin Tang wrote: > > > > diff --git a/drivers/gpu/drm/sprd/Makefile > > > b/drivers/gpu/drm/sprd/Makefile > > > > index 6c25bfa99..d49f4977b 100644 > > > > --- a/drivers/gpu/drm/sprd/Makefile > > > > +++ b/drivers/gpu/drm/sprd/Makefile > > > > @@ -1,5 +1,8 @@ > > > > # SPDX-License-Identifier: GPL-2.0 > > > > > > > > obj-y := sprd_drm.o \ > > > > - sprd_dpu.o > > > > - > > > > + sprd_dpu.o \ > > > > + sprd_dsi.o \ > > > > + dw_dsi_ctrl.o \ > > > > + dw_dsi_ctrl_ppi.o \ > > > > > > So it's a designware IP? There's a driver for it already that seems > > > fairly similar: > > > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c > > > > > Our dw dsi controller is not a standard synopsys ip, we have updated a lot > > on the basic ip version, > > the entire control register is different, i have cc to drm/bridge reviewers > > and maintainers. > > You should make it more obvious then in a comment or in the name of the > driver. If it's fairly different from the original IP from Synopsys, > maybe you should just drop the reference to the name? Thks, i will be fix it. > > Maxime _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2021-04-15 8:20 UTC|newest] Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-22 13:28 [PATCH v4 0/6] Add Unisoc's drm kms module Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-02-22 13:28 ` [PATCH v4 1/6] dt-bindings: display: add Unisoc's drm master bindings Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-02-22 13:28 ` [PATCH v4 2/6] drm/sprd: add Unisoc's drm kms master Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-02-22 20:55 ` Orson Zhai 2021-02-22 20:55 ` Orson Zhai 2021-03-21 19:26 ` Kevin Tang 2021-03-24 10:53 ` Maxime Ripard 2021-03-24 10:53 ` Maxime Ripard 2021-03-31 1:20 ` Kevin Tang 2021-04-08 10:21 ` Thomas Zimmermann 2021-04-08 10:21 ` Thomas Zimmermann 2021-04-09 13:50 ` Kevin Tang 2021-04-09 13:57 ` Thomas Zimmermann 2021-04-09 13:57 ` Thomas Zimmermann 2021-02-22 13:28 ` [PATCH v4 3/6] dt-bindings: display: add Unisoc's dpu bindings Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-02-22 13:28 ` [PATCH v4 4/6] drm/sprd: add Unisoc's drm display controller driver Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-03-24 11:10 ` Maxime Ripard 2021-03-24 11:10 ` Maxime Ripard 2021-03-31 1:21 ` Kevin Tang 2021-04-07 10:45 ` Maxime Ripard 2021-04-07 10:45 ` Maxime Ripard 2021-04-08 3:05 ` Chunyan Zhang 2021-04-08 3:05 ` Chunyan Zhang 2021-04-09 13:35 ` Kevin Tang 2021-04-15 9:09 ` Maxime Ripard 2021-04-15 9:09 ` Maxime Ripard 2021-04-15 0:18 ` Kevin Tang 2021-04-15 0:18 ` Kevin Tang 2021-04-15 9:03 ` Maxime Ripard 2021-04-15 9:03 ` Maxime Ripard 2021-04-18 23:01 ` Kevin Tang 2021-04-18 23:01 ` Kevin Tang 2021-04-21 10:00 ` Maxime Ripard 2021-04-21 10:00 ` Maxime Ripard 2021-04-08 10:53 ` Thomas Zimmermann 2021-04-08 10:53 ` Thomas Zimmermann 2021-04-12 4:44 ` Kevin Tang 2021-02-22 13:28 ` [PATCH v4 5/6] dt-bindings: display: add Unisoc's mipi dsi controller bindings Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-03-24 11:13 ` Maxime Ripard 2021-03-24 11:13 ` Maxime Ripard 2021-03-31 1:49 ` Kevin Tang 2021-04-07 10:46 ` Maxime Ripard 2021-04-07 10:46 ` Maxime Ripard 2021-04-09 0:23 ` Kevin Tang 2021-04-15 8:42 ` Maxime Ripard 2021-04-15 8:42 ` Maxime Ripard 2021-04-18 16:33 ` Kevin Tang 2021-04-18 16:33 ` Kevin Tang 2021-04-21 9:38 ` Maxime Ripard 2021-04-21 9:38 ` Maxime Ripard 2021-02-22 13:28 ` [PATCH v4 6/6] drm/sprd: add Unisoc's drm mipi dsi&dphy driver Kevin Tang 2021-02-22 13:28 ` Kevin Tang 2021-03-24 11:27 ` Maxime Ripard 2021-03-31 1:47 ` Kevin Tang 2021-04-07 10:48 ` Maxime Ripard 2021-04-07 10:48 ` Maxime Ripard 2021-04-15 0:19 ` Kevin Tang [this message] 2021-04-15 0:19 ` Kevin Tang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAFPSGXbJwNd47UAwVu4c1t5K7j4sgMtTLtDZ15woHZbRTghVfA@mail.gmail.com \ --to=kevin3.tang@gmail.com \ --cc=airlied@linux.ie \ --cc=daniel@ffwll.ch \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=linux-kernel@vger.kernel.org \ --cc=maarten.lankhorst@linux.intel.com \ --cc=mark.rutland@arm.com \ --cc=maxime@cerno.tech \ --cc=orsonzhai@gmail.com \ --cc=robh+dt@kernel.org \ --cc=sean@poorly.run \ --cc=zhang.lyra@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.