Hi Sjur,
>> +/* HACK: for temporary backwards compatibility */
>> +#define hangup_all hangup
>> + void (*hangup_all)(struct ofono_voicecall *vc,
>> + ofono_voicecall_cb_t cb, void *data);
>> + void (*hangup_active)(struct ofono_voicecall *vc,
>
> We'll need to figure out how to migrate to the new driver API a bit more
> sanely.
Should we s/hangup/hangup_active in all drivers and skip this hack then?
Or did you have something else in mind?
That sounds good. Break this up into two patches, one renaming hangup
to hangup_active and the second one introducing hangup_all.
How do we proceede with this change. I am very happy if you carry on from here,
or would you like me to to submit a patch-set with these changes,
including s/hangup/hangup_active in drivers/*/voicecall.c?
Go ahead and re-submit the fixed up patch set and I will take care of
the rest.
Regards,
-Denis