From: Chris Morgan <macroalpha82@gmail.com>
To: Robert Foss <robert.foss@linaro.org>
Cc: andrzej.hajda@intel.com, narmstrong@baylibre.com,
Laurent.pinchart@ideasonboard.com, jonas@kwiboo.se,
jernej.skrabec@gmail.com, airlied@linux.ie, daniel@ffwll.ch,
robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
dianders@chromium.org, lkundrak@v3.sk,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Chris Morgan <macromorgan@hotmail.com>,
tzimmermann@suse.de, javierm@redhat.com,
maarten.lankhorst@linux.intel.com, mripard@kernel.org
Subject: Re: [PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series
Date: Mon, 12 Sep 2022 16:35:47 -0500 [thread overview]
Message-ID: <20220912213547.GA6141@wintermute.localdomain> (raw)
In-Reply-To: <20220912113856.817188-1-robert.foss@linaro.org>
On Mon, Sep 12, 2022 at 01:38:54PM +0200, Robert Foss wrote:
> After applying the "chrontel-ch7033: Add byteswap order option" series,
> Laurent reported an issues with the approach. Since no fix has been submitted
> for the issues outlined in time for the next kernel release, I'd like to
> revert this series for now.
>
> Just to be clear I would very much like to see a v3 of this[1] series, where the
> issues outlined have been fixed.
I will work on a v3 soon, I just have to finish a few other things first.
That said, I'm not very familiar with what we're trying to do in an automated
fashion. In my use case I have DPI output (from an Allwinner R8 to this bridge)
which then connects via HDMI. I'm aware that we should be able to get the color
space information from the HDMI connector, correct? Is it that information I
would then need to use to set the bridge colorspace, or is it the color info
from the DPI connector I'm using?
I'm still pretty new to DRM drivers so this is mostly new to me. Thank you.
>
> [1] https://lore.kernel.org/all/20220902153906.31000-1-macroalpha82@gmail.com/
>
> Robert Foss (2):
> Revert "dt-bindings: Add byteswap order to chrontel ch7033"
> Revert "drm/bridge: ti-sn65dsi86: Implement bridge connector
> operations for DP"
>
> .../display/bridge/chrontel,ch7033.yaml | 13 ---------
> drivers/gpu/drm/bridge/ti-sn65dsi86.c | 28 -------------------
> 2 files changed, 41 deletions(-)
>
> --
> 2.34.1
>
prev parent reply other threads:[~2022-09-12 21:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-12 11:38 [PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series Robert Foss
2022-09-12 11:38 ` [PATCH v1 1/2] Revert "dt-bindings: Add byteswap order to chrontel ch7033" Robert Foss
2022-09-13 15:49 ` Rob Herring
2022-09-12 11:38 ` [PATCH v1 2/2] Revert "drm/bridge: ti-sn65dsi86: Implement bridge connector operations for DP" Robert Foss
2022-09-12 14:29 ` Doug Anderson
2022-09-12 14:42 ` Laurent Pinchart
2022-09-15 10:32 ` Robert Foss
2022-09-12 12:33 ` [PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series Laurent Pinchart
2022-09-12 21:35 ` Chris Morgan [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=20220912213547.GA6141@wintermute.localdomain \
--to=macroalpha82@gmail.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@linux.ie \
--cc=andrzej.hajda@intel.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=javierm@redhat.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkundrak@v3.sk \
--cc=maarten.lankhorst@linux.intel.com \
--cc=macromorgan@hotmail.com \
--cc=mripard@kernel.org \
--cc=narmstrong@baylibre.com \
--cc=robert.foss@linaro.org \
--cc=robh+dt@kernel.org \
--cc=tzimmermann@suse.de \
/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