[Ovmsdev] Refactoring MODEM/SIMCOM for v3.3
mark at webb-johnson.net
Sun Aug 30 18:15:58 HKT 2020
“Cellular” sounds good. I will make the change. “mod<tab>” has been aggravating me.
P.S. Still working on it, but hopefully SIM5360 is working well.
> On 30 Aug 2020, at 5:39 PM, Michael Balzer <dexter at expeedo.de> wrote:
> Running smoothly on my bench module.
> Your refactorization changes also look good to me. (Side note: I'd opt for generally keeping trailing white space. I use them for graphical purposes and to denote continuing paragraphs in text files (rst), which my editor automatically does to provide consistent rewrapping.)
> I suggest renaming the command root "modem" to something not breaking the "mo" or "mod" shortcut for "module". I generally try to make/keep command abbreviations work at 2-3 characters.
> As "modem" is also a general device class (also applicable to the wifi modem), how about calling it "cellular" instead?
> That would also allow to add some general cellular network specific commands without breaking semantics.
> Am 30.08.20 um 09:20 schrieb Mark Webb-Johnson:
>> The initial implementation of this work is in branch for-v3.3, in GitHub. That branch can be used for the upcoming 3.3 release, including potentially breaking changes.
>> I’d appreciate any feedback. For the SIM5360, the behaviour should be unchanged from the current implementation (except some improvements to edge cases and control logic). The major user visible change is from ’simcom’ to ‘modem’ for the console commands.
>> Regards, Mark.
>>> On 20 Aug 2020, at 9:35 PM, Mark Webb-Johnson <mark at webb-johnson.net> wrote:
>>> I have started the work on refactoring the modem driver to support virtual driver implementations (SIMCOM 5360 being just one of them). This is a major refactoring that will take some time to complete. It is likely that the following components/files will be affected:
>>> Component simcom (major changes)
>>> New component ovms_modem introduced
>>> Component powermgmt
>>> And several other components mentioning Simcom, but only very minor changes
>>> I would appreciate it if no major changes were made to those components in the next week or two, as that may make merging difficult.
>>> Once I’ve got the modem code refactored, and a compatible SIMCOM 5360 driver working, I’ll publish to the for-v3.3 branch for wider testing. I can then add the other modem drivers I have been working on.
>>> Regards, Mark.
>>> P.S. Pretty obvious that this work is to abstract out the modem type specific code, to make it easier to add support for other modem types in OVMS. This is a long-term project, and not something happening anytime soon, but I am working on it.
>>> OvmsDev mailing list
>>> OvmsDev at lists.openvehicles.com
>> OvmsDev mailing list
>> OvmsDev at lists.openvehicles.com
> Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
> Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
> OvmsDev mailing list
> OvmsDev at lists.openvehicles.com
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OvmsDev