mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Shawn Sung (宋孝謙)" <Shawn.Sung@mediatek.com>
To: "CK Hu (胡俊光)" <ck.hu@mediatek.com>,
	"chunkuang.hu@kernel.org" <chunkuang.hu@kernel.org>,
	"angelogioacchino.delregno@collabora.com"
	<angelogioacchino.delregno@collabora.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mediatek@lists.infradead.org" 
	<linux-mediatek@lists.infradead.org>,
	"daniel@ffwll.ch" <daniel@ffwll.ch>,
	"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"seanpaul@chromium.org" <seanpaul@chromium.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"airlied@gmail.com" <airlied@gmail.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>
Subject: Re: [PATCH v3 1/1] drm/mediatek: Fix errors when reporting rotation capability
Date: Fri, 24 Nov 2023 09:41:16 +0000	[thread overview]
Message-ID: <8caf6b3f146850636a7decdedf487e5dc067de18.camel@mediatek.com> (raw)
In-Reply-To: <e7316d95958f9628186e8768f10f691657ea77a1.camel@mediatek.com>

Hi CK,

On Tue, 2023-11-21 at 02:57 +0000, CK Hu (胡俊光) wrote:
> Hi, Hsiao-chien:
> 
> On Mon, 2023-11-20 at 15:12 +0800, Hsiao Chien Sung wrote:
> > Create rotation property according to the hardware capability.
> > 
> > Fixes: 84d805753983 ("drm/mediatek: Support reflect-y plane
> > rotation")
> 
> In Sean Paul's patch (the patch you fix), ovl driver only support
> mt8127 and mt8173, and I believe that Sean has tested on mt8173. In
> this patch, only mt8195 ovl support rotation, why do you do this
> modification? Please describe which SoC is wrong in commit message so
> I
> could know how the code should be.

Thanks for checking. According to the cover letter by Sean, his fix was
verified on mt8183, but it will be applied to all the chips. Since for
now OVL on all chips support same rotation, will remove it from the
driver data and return rotation property directly.

Regards,
Shawn

      reply	other threads:[~2023-11-24  9:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-20  7:12 [PATCH v3 0/1] " Hsiao Chien Sung
2023-11-20  7:12 ` [PATCH v3 1/1] drm/mediatek: " Hsiao Chien Sung
2023-11-21  2:57   ` CK Hu (胡俊光)
2023-11-24  9:41     ` Shawn Sung (宋孝謙) [this message]

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=8caf6b3f146850636a7decdedf487e5dc067de18.camel@mediatek.com \
    --to=shawn.sung@mediatek.com \
    --cc=airlied@gmail.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=chunkuang.hu@kernel.org \
    --cc=ck.hu@mediatek.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=p.zabel@pengutronix.de \
    --cc=seanpaul@chromium.org \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®