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

[Matthias Brugger]

On 05/11/16 00:21, Daniel Kurtz wrote:
> On Tue, Oct 25, 2016 at 6:23 AM, Matthias Brugger
> <matthias.bgg@xxxxxxxxx> 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.

After checking other users of request_firmware and check them against 
linux-firmware I think we don't need the FW in linux-firmware to get the 
driver merged. Especially as there already is a working FW stored on the 
device.

Regards,
Matthias