linux-kernel - Re: [PATCH v2 11/18] drm/sun4i: Add A83T support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 28 Nov 2017 19:50:19 +0800
From:   Icenowy Zheng <icenowy@...c.io>
To:     Maxime Ripard <maxime.ripard@...e-electrons.com>
Cc:     Jernej Škrabec <jernej.skrabec@...l.net>,
        Daniel Vetter <daniel.vetter@...el.com>,
        David Airlie <airlied@...ux.ie>, Chen-Yu Tsai <wens@...e.org>,
        dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
        Mark Rutland <mark.rutland@....com>,
        Rob Herring <robh+dt@...nel.org>,
        linux-arm-kernel@...ts.infradead.org, linux-clk@...r.kernel.org,
        plaes@...es.org,
        Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
Subject: Re: [PATCH v2 11/18] drm/sun4i: Add A83T support

在 2017-11-28 17:02，Maxime Ripard 写道：
> Hi,
> 
> On Mon, Nov 27, 2017 at 05:01:49PM +0100, Jernej Škrabec wrote:
>> Dne ponedeljek, 27. november 2017 ob 16:41:35 CET je Maxime Ripard 
>> napisal(a):
>> > Add support for the A83T display pipeline.
>> >
>> > Reviewed-by: Chen-Yu Tsai <wens@...e.org>
>> > Signed-off-by: Maxime Ripard <maxime.ripard@...e-electrons.com>
>> > ---
>> >  Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 3 +++
>> >  drivers/gpu/drm/sun4i/sun4i_drv.c                             | 2 ++
>> >  drivers/gpu/drm/sun4i/sun4i_tcon.c                            | 5 +++++
>> >  drivers/gpu/drm/sun4i/sun8i_mixer.c                           | 4 ++++
>> >  4 files changed, 14 insertions(+)
>> >
>> > diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt
>> > b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt index
>> > d4259a4f5171..d6b52e5c48c0 100644
>> > --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt
>> > +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt
>> > @@ -93,6 +93,7 @@ Required properties:
>> >     * allwinner,sun6i-a31s-tcon
>> >     * allwinner,sun7i-a20-tcon
>> >     * allwinner,sun8i-a33-tcon
>> > +   * allwinner,sun8i-a83t-tcon-lcd
>> >     * allwinner,sun8i-v3s-tcon
>> >   - reg: base address and size of memory-mapped region
>> >   - interrupts: interrupt associated to this IP
>> > @@ -224,6 +225,7 @@ supported.
>> >
>> >  Required properties:
>> >    - compatible: value must be one of:
>> > +    * allwinner,sun8i-a83t-de2-mixer
>> 
>> What will be the name of the second mixer, once support for HDMI is 
>> added?
>> Should we start directly with 0 and 1 postfix ?
> 
> What are the differences exactly without the two mixers?
> 
> I was hoping to be able to cover them all using properties, indices
> are usually pretty badly received in compatibles.

1. VEP is only available on the VI channel in mixer0. (VEP may mean
Video Enhance Processor)
2. Smart Backlight is only available in mixer0's on SoCs with LCD.
3. Writeback function is only available in mixer0.

> 
>> >  static const struct of_device_id sun8i_mixer_of_table[] = {
>> >  	{
>> > +		.compatible = "allwinner,sun8i-a83t-de2-mixer",
>> > +		.data = &sun8i_v3s_mixer_cfg,
>> > +	},
>> > +	{
>> 
>> Maybe you want to squash 12 patch since this works only by luck.
> 
> Yeah, I totally meant to do that :)
> 
> Thanks!
> Maxime