Hi,
2011/1/25 Marcel Holtmann <mar...@holtmann.org>:
>> +enum isi_version {
>> + ISI_20 = 0,
>> + ISI_25 = 1
>> +};
>> +
>
> If we follow the same layout as with all other modem drivers, then the
> *modem.h should only contain the prototypes for the init callbacks.
>
>> extern struct ofono_modem *isi_modem_by_interface(char const *ifname);
>
> This one already violates this one and might need to be cleaned up.
>
> The ISI modem is special since it really only has two users anyway in
> the end, but maybe drivers/isimodem/version.h analog to vendor.h from
> atmodem might be a good idea.
>
> Aki, what is your preference here?
I don't think there is a need for this, as ISI itself already provides
both ISI version (which we have available always via the
g_isi_client_verify() call), and in many cases also the resource ID as
means to find the correct quirk to use.
Of course, I could be convinced otherwise, but not without seeing
actual patches. It might become extremely ugly with using ISI version
and the resource ID, and using something like vendor.h might then make
sense.
But remember that we also have the isiusb plugin, and I would like it
to keep working with *any* isimodem out there.
Cheers,
Aki
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono
