Cannot send message: QMI service 'pdc' version '1.15' required, got version '1.0'

[Dan Williams]

On Fri, 2019-12-13 at 18:37 +0100, Aleksander Morgado wrote:
> > > I’m testing mm 1.12.2 + libqmi 1.24.2 on sierra EM7565 (openwrt).
> > > Please see last line of log. Is this ok?
> > > 
> > > 
> > > 
> > > I do not see this in mm 1.12.0 + libqmi 1.24.0
> > > 
> > > 
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Enabling QMI
> > > indications via MBIM...
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] enabled QMI
> > > indications via MBIM
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: <info>  [cdc-wdm0] MBIM device is
> > > QMI capable
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Allocating new
> > > client ID...
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Registered 'dms'
> > > (version 1.0) client with ID '1'
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Allocating new
> > > client ID...
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Registered 'nas'
> > > (version 1.25) client with ID '2'
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Allocating new
> > > client ID...
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Registered 'loc'
> > > (version 2.0) client with ID '1'
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Allocating new
> > > client ID...
> > > 
> > > Dec 13 15:51:38 OpenWrt [5904]: [/dev/cdc-wdm0] Registered 'pdc'
> > > (version 1.0) client with ID '1'
> > > 
> > > Dec 13 15:51:39 OpenWrt [5904]: <info>  QMI-based capability and
> > > mode switching support enabled
> > > 
> > > Dec 13 15:51:39 OpenWrt [5904]: <warn>  couldn't load carrier
> > > config: 'Cannot send message: QMI service 'pdc' version '1.15'
> > > required, got version '1.0''
> > > 
> > 
> > I believe the issue is unrelated to the MM/libqmi version. It would
> > be
> > very strange that the modem replies differnet things when using
> > different libqmi versions really.
> > 
> > The problem, though, is something we should fix. The version checks
> > done by libqmi are truly broken, because they rely on the version
> > info
> > for each message that is defined in our message database, and that
> > information is far from good or up to date. If the message is not
> > supported by the device, we'll get an error when trying to use it.
> > Trying to return an early error before sending the request assuming
> > our information of when it was introduced is good is not the way to
> > go.
> > 
> 
> See 
> https://gitlab.freedesktop.org/mobile-broadband/libqmi/merge_requests/80
> 
> Comments welcome

I see you merged it 6 hours ago, but FWIW it looks good to me :)

Dan

>