On 23/1/21 10:52 pm, Gerhard Sittig wrote:
On Sat, 2021-01-23 at 08:43 +1100, Peter Skarpetis wrote:
I have recently added support for the MHINSTEK MHS-5200A function
generator to libsigrok and have created a pull request for you to
merge.
https://github.com/sigrokproject/libsigrok/pull/113
I have tested it with an MHS-5225A but should work with other models
as well.
That's not mergable in its current form. Can you do some cleanup?
Introduce the files with the proper names from the start. Move
the device communication to protocol.c and keep api.c as they are
for all other drivers. Fixup style issues. Consider squashing the
parts because: From the project's perspective this is the
introduction of an initial implementation of support for this
device (the first working version which covers a basic set of
features or all those features which translate to sigrok
infrastructure). Doesn't matter how many steps it took to get
there during internal development at your place. Consider the
perspective of those who integrate your contribution. Unless the
commits' activities really are separate things from a maintenance
perspective.
I have cleaned up the code as requested and have created a new patchset
that you can access using the following link. This is a single commit as
requested.
https://github.com/peterska/libsigrok/commit/3c4ca8764a847025d624ddb2d5ac7d6896b3dc49.patch
Can you estimate how much the MHS5200A and the JDS6600 devices'
protocols differ. You stated they do. Is it "just a little" or
"totally so"? Am asking since there is the question whether one
driver could support several dialects or minor variations, or
whether separate drivers are required since differences are too
many or too severe.
(We can also discuss this in IRC when you are around there.
Results in a more fluent conversation than mail exchange.)
I do not have a JDS6600 myself, the protocol is similar although it uses
a different command structure to achieve the same thing.
I am happy to add the commands for the JDS6600 as long as someone else
can test it.
I am currently at the Linux Australia conference and don't have time for
a irc chat.
Do let me know if I can do anymore changes before the patch is ready.
Virtually yours,
Peter Skarpetis
Who else has got devices of that MHS series and can test?
virtually yours
Gerhard Sittig
--
If you don't understand or are scared by any of the above
ask your parents or an adult to help you.
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel
