[PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series

Robert Foss posted 2 patches 3 years, 6 months ago
.../display/bridge/chrontel,ch7033.yaml       | 13 ---------
drivers/gpu/drm/bridge/ti-sn65dsi86.c         | 28 -------------------
2 files changed, 41 deletions(-)
[PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series
Posted by Robert Foss 3 years, 6 months ago
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.

[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
Re: [PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series
Posted by Laurent Pinchart 3 years, 6 months ago
Hi Rob,

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.
> 
> [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"

For the series,

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

>  .../display/bridge/chrontel,ch7033.yaml       | 13 ---------
>  drivers/gpu/drm/bridge/ti-sn65dsi86.c         | 28 -------------------
>  2 files changed, 41 deletions(-)

-- 
Regards,

Laurent Pinchart
Re: [PATCH v1 0/2] Revert chrontel-ch7033 byteswap order series
Posted by Chris Morgan 3 years, 6 months ago
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
>