linux-kernel - Re: [v17 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

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]

Message-ID: <CAGS+omDmFdqzuFqcCe9gFGemZc88Yt0_3_o2KwOgW=GN_kCakQ@mail.gmail.com>
Date:   Fri, 4 Nov 2016 17:21:14 -0600
From:   Daniel Kurtz <djkurtz@...omium.org>
To:     Matthias Brugger <matthias.bgg@...il.com>
Cc:     Enric Balletbo Serra <eballetbo@...il.com>,
        Jitao Shi <jitao.shi@...iatek.com>,
        David Airlie <airlied@...ux.ie>,
        Thierry Reding <treding@...dia.com>,
        Mark Rutland <mark.rutland@....com>, stonea168@....com,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        Andy Yan <andy.yan@...k-chips.com>,
        Ajay Kumar <ajaykumar.rs@...sung.com>,
        Vincent Palatin <vpalatin@...omium.org>,
        cawa cheng <cawa.cheng@...iatek.com>,
        Bibby Hsieh (謝濟遠) 
        <bibby.hsieh@...iatek.com>,
        CK Hu (胡俊光) <ck.hu@...iatek.com>,
        Russell King <rmk+kernel@....linux.org.uk>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        Sascha Hauer <kernel@...gutronix.de>,
        Pawel Moll <pawel.moll@....com>,
        Ian Campbell <ijc+devicetree@...lion.org.uk>,
        Inki Dae <inki.dae@...sung.com>,
        Rob Herring <robh+dt@...nel.org>,
        "moderated list:ARM/Mediatek SoC support" 
        <linux-mediatek@...ts.infradead.org>,
        Yingjoe Chen <yingjoe.chen@...iatek.com>,
        Eddie Huang (黃智傑) 
        <eddie.huang@...iatek.com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        Rahul Sharma <rahul.sharma@...sung.com>,
        srv_heupstream <srv_heupstream@...iatek.com>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Kumar Gala <galak@...eaurora.org>,
        Sean Paul <seanpaul@...omium.org>
Subject: Re: [v17 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

On Tue, Oct 25, 2016 at 6:23 AM, Matthias Brugger
<matthias.bgg@...il.com> wrote:
>
> On 10/18/2016 04:37 PM, Enric Balletbo Serra wrote:
> [...]
>>> --- /dev/null
>>> +++ b/drivers/gpu/drm/bridge/parade-ps8640.c
> [...]
>>>
>>> +
>>> +/* Firmware */
>>> +#define PS_FW_NAME             "ps864x_fw.bin"
>>> +
>>
>> From where I can download this firmware image?
>
> I suppose this FW bits have to be added to linux-firmware repository first, before this patch can be accepted.

All PS8640 devices should already ship with working firmware.
The firmware update procedure is only used in the unlikely event where
one wants to update the bridge to a different firmware provided by
Parade.

Why must the lack of firmware really block landing this driver?

If this is really so, can we just land the functional part of the
driver first, and add the firmware update in a follow-up patch.

>
> Regards,
> Matthias