summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add changes file for 5.9.1v5.9.1Alex Blasche2017-06-221-0/+30
| | | | | Change-Id: I40fa19b016ca5e8e8e831a909eb378545a026446 Reviewed-by: André Hartmann <aha_1980@gmx.de>
* Fix build with Clang 5.0: remove unused captureThiago Macieira2017-06-131-1/+1
| | | | | | | qmodbusrtuserialmaster_p.h:356:68: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture] Change-Id: Ia53158e207a94bf49489fffd14c7903c7e4655cb Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Merge remote-tracking branch 'origin/5.9.0' into 5.9Liang Qi2017-05-312-0/+63
|\ | | | | | | Change-Id: I9c93f053c5a6a7af04c79d6e2cc8ded76c906319
| * fix namespaced buildv5.9.0-rc2v5.9.0-rc1v5.9.0Oswald Buddenhagen2017-05-121-0/+2
| | | | | | | | | | Change-Id: I7daa9be5b0b21c0f94021b9a2eb139915c501049 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * Add changes file for 5.9.0Alex Blasche2017-05-101-0/+61
| | | | | | | | | | | | | | | | Listing important changes and bug-fixes. Change-Id: I2f3b131c319feb0e95d8d08e9a07b71078f52a57 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: André Hartmann <aha_1980@gmx.de>
* | CAN Example: Add menu entry to show online documentationAndre Hartmann2017-05-292-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | The different plugins have varying runtime requirements, that should be described in the plugins documentation. Allow the user to open the documentation easily. [ChangeLog][QCanBus] Added a menu to open plugins online documentation to the CAN Example. Change-Id: I2ae19b439f91042c6bcb09a7db079249fb34355c Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | PeakCAN: Update supported channels documentationAndre Hartmann2017-05-231-2/+2
| | | | | | | | | | | | | | | | Amends commit 3f8eb3be91641f9dba597e4421f9bbf50c5c161e Change-Id: Ibc0a1019ac4a4e63044c43125826614983a604e4 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | CAN: Add requirements to plugin documentationAndre Hartmann2017-05-234-0/+12
| | | | | | | | | | | | Change-Id: I0dbbf10d8448c5d26f28bd7fa84cdeab21e6eb5c Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | Bump versionOswald Buddenhagen2017-05-101-1/+1
|/ | | | Change-Id: I65aebb9442ff23158f29068e6f4a789217af99be
* VectorCAN: Fix crash when device open failsv5.9.0-beta4Andre Hartmann2017-04-251-0/+6
| | | | | | | | | | | | | | | | | | | | xlOpenPort() fails to open "can0" when the Vector vxlapi.dll is available, but no Vector device driver is installed. Calling xlClosePort() afterwards crashes. Before this patch, xlDeactivateChannel() and xlClosePort() were called unconditionally as part of VectorCanBackendPrivate::close() when any VectorCanBackendPrivate::open() step failed. Now xlClosePort() is only called when xlOpenPort() succeeded. Also guard xlDeactivateChannel() for consistence (doesn't make sense to call it on a closed device), even if this never crashed for me. Change-Id: Iad480f7e02ca8f45b8cc4d1d8930640b2d8cbeb2 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Fix build with -no-feature-commandlineparserStephan Binner2017-04-251-1/+1
| | | | | Change-Id: Ie939d23f1de0956fa34c74f52180775af8d38524 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* QCanBusDeviceInfo: Remove unused forward declarationAndre Hartmann2017-04-251-1/+0
| | | | | | | | Looks like a relict from a former implementation. Change-Id: Ic8c1c81151bc7877a991d35d410680f9bc29448a Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* CAN: Remove unneded core-private dependency for pluginsv5.9.0-beta3v5.9.0-beta2Andre Hartmann2017-04-135-5/+5
| | | | | | Change-Id: I9f5c41a5c3cd0c83f3835d29de22b76ac367cdbb Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* VectorCAN: Un-wrap medium long linesAndre Hartmann2017-04-121-10/+5
| | | | | | | | All lines are still shorter than 100 chars and even shorter as the longest line in this file. Change-Id: I227b13f78b3b7628287729f40ffec7bfa06231db Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* VectorCAN: Fix error category for disconnection errorAndre Hartmann2017-04-121-1/+1
| | | | | | Change-Id: Ic3dce0c8b91e9994c31763725de48f4900fd9bef Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
* QCanBusDeviceInfo: Improve implementationAndre Hartmann2017-04-083-27/+15
| | | | | | | | | | | | | | | | As pointed out in the 5.9 API review. Deleted the standard constructor to not assign memory. The only way to construct QCanBusDeviceInfo is now by using the return value of QCanBus::availableDevices(): const auto infos = QCanBus::instance()->availableDevices(backend); for (const QCanBusDeviceInfo &info : infos) // do something with info Change-Id: I3382d8a8fffadd18e4f2f87d4ede94e7c2e442f2 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* QCanBusFrame: Add missing Error State Indicator streaming testsAndre Hartmann2017-04-041-9/+17
| | | | | | | | Amends commit 19d4d81e8dd7ccb9831bea725989d2a08d258207 Change-Id: Icd9e64c0b480b627e22b3d4fce4fff68a87ab410 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Rolf Eike Beer <eb@emlix.com>
* canbusutil: Remove unused declarationsAndre Hartmann2017-04-032-4/+0
| | | | | Change-Id: Ib3f7925f0f063c12bea487c73f8cb1c5817c2492 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* Merge remote-tracking branch 'origin/5.8' into 5.9v5.9.0-beta1Liang Qi2017-03-31112-129/+4888
|\ | | | | | | | | | | | | | | Conflicts: src/plugins/canbus/canbus.pro src/plugins/canbus/vectorcan/vectorcanbackend.cpp Change-Id: I3625693784e88204c93479806242ae70c7ec7ff3
| * Fix QModbusRtuSerialMaster wouldn't close5.8Konstantin Shtepa2017-03-301-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | QModbusRtuSerialMaster wouldn't close on disconnect and would never release serial port because of patch in QTBUG-56009. Revert part of patch 6bed30d5ceef1463469e37173b3ebac2cfdc2352 which lead to this state. Task-number: QTBUG-59050 Change-Id: I8b3280a2889a416aa80e7f58264b5f9696b8c8b9 Reviewed-by: André Hartmann <aha_1980@gmx.de> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * Revert "QCanBusFactory: de-inline virtual destructor"André Hartmann2017-03-272-4/+4
| | | | | | | | | | | | | | | | | | | | | | It turned out, that this change is not binary compatible as it was meant to be. Loading the plugins with QPluginLoader failed, so this patch needs to be reverted. This reverts commit b63a833346ed14baddbf598d48e005090b8bcb56. Change-Id: I32c7e3a46bdd5e2ea6217e6d2050a3878f53f1ff Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * modbus master example: fix nonsensical codeMarc Mutz2017-03-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Pointed out by GCC 7: writeregistermodel.cpp:117:48: warning: enum constant in boolean context [-Wint-in-bool-context] if (index.column() == HoldingColumn && Qt::EditRole) { // holding registers ^~~~~~~~ Change-Id: I5d0f19c0480b027109c6a751f69852114cb634b2 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: David Faure <david.faure@kdab.com>
| * VectorCAN: Fix build with -no-feature-libraryAndre Hartmann2017-03-091-2/+1
| | | | | | | | | | | | | | Amends commit 6803fbb01566e74df895326fb7805868696ce9c4 Change-Id: I97d06afa554df0c7528164f8accf9196c542e65c Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
| * Fix build with -no-feature-libraryUlf Hermann2017-03-063-2/+6
| | | | | | | | | | | | | | | | | | Drop an unused include, don't try to set the library path if !QT_CONFIG(library) and don't build the peakcan and tinycan plugins as those use QLibrary to look up their symbols. Change-Id: Id8b1245c1cf69f7548e050d47d5956f53562477d Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * QCanBusFactory: de-inline virtual destructorAndre Hartmann2017-03-032-4/+4
| | | | | | | | | | | | | | | | As demanded in the API review. Change-Id: I640ab16a12ab9297270371550236eda903f26ea2 Reviewed-by: Marc Mutz <marc.mutz@kdab.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * VectorCAN: Load vxlapi64.dll for 64 bit buildsAndre Hartmann2017-02-171-0/+4
| | | | | | | | | | | | | | Task-number: QTBUG-58774 Change-Id: I9ac12dbb5ae267a063b3c60130ed86499bce22dc Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * VectorCAN: Fix receiving frames with extended IDAndre Hartmann2017-02-162-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Beside the CAN ID, the bit XL_CAN_EXT_MSG_ID is stored in event.tagData.msg.id. This bit indicates, that the received frame has extended frame format. However, this bit must be masked before calling QCanBusFrame::setFrameId(), otherwise the frame is marked as invalid with the ID set to zero. Most likely this was broken by commit a54fd820916caed2fb13b1a38e57deb5d918f74f Task-number: QTBUG-58840 Change-Id: I1814f19fe3ba8a7d057c42e112b62228fabf4206 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * Binary compatibility file and global.cfg for qtserialbus (5.8.0)Milla Pohjanheimo2017-02-062-0/+4685
| | | | | | | | | | | | | | Added binary compatibility file and global.cfg for qtserialbus Change-Id: I43c27c5ce36718856a1b3fa7b8870c3de003d819 Reviewed-by: Sergio Ahumada <sahumada@texla.cl>
| * Updated copyright headersAndre Hartmann2017-02-02108-118/+118
| | | | | | | | | | Change-Id: I7d38491c7ee15a1de788375acf9144c59b507d2a Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| * Merge remote-tracking branch 'origin/5.8.0' into 5.8Liang Qi2017-01-262-1/+70
| |\ | | | | | | | | | Change-Id: I34693c3a4b342b1a5b30b5adc4af34f67fccb8b2
| | * Prettify changes file for 5.8.0v5.8.0-rc1v5.8.0Andre Hartmann2016-12-121-6/+6
| | | | | | | | | | | | | | | Change-Id: Ieb5bd937787c2f68f97c9ef505e6a0ca091806ac Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
| | * Fix ICC warning about conversion changing signThiago Macieira2016-12-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | error #68: integer conversion resulted in a change of sign QByteArray elements are char, not quint8. Change-Id: I73fa1e59a4844c43a109fffd148caecfb59c95cc Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: André Hartmann <aha_1980@gmx.de>
| | * Add changes file for 5.8.0Alex Blasche2016-12-071-0/+69
| | | | | | | | | | | | | | | Change-Id: I46a0970cefa87d2fd401b74638f80c3395ff2391 Reviewed-by: André Hartmann <aha_1980@gmx.de>
* | | QCanBus: Add convenience constructor to QCanBusPrivateAndre Hartmann2017-03-311-4/+2
| | | | | | | | | | | | | | | Change-Id: I1a63fc5ceef54e2c8fb3d853990d8e0370f940b5 Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
* | | QCanBus: Use QLatin1String overload of QJsonObject::value() in loadPluginsAndre Hartmann2017-03-301-2/+2
| | | | | | | | | | | | | | | | | | | | | The overload was introduced in Qt 5.7. Change-Id: I99ae1bd0ef896213b6c5a85277ff0dfb1da1602a Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
* | | canbusutil: Print available devices for given pluginAndre Hartmann2017-03-303-4/+26
| | | | | | | | | | | | | | | | | | Change-Id: I72d2dd90137c3be2e07311d1dc4f83dfa4c15a18 Reviewed-by: Rolf Eike Beer <eb@emlix.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | canbusutil: Better output formatting with timestamp and infoAndre Hartmann2017-03-301-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Print two spaces after the info to separate the CAN ID, print no space before the info as the timestamp already ends with two spaces. Example output: 1490815444.0159 - - 12345678 [04] 00 11 22 33 1490815444.0171 B - 12345678 [04] 00 11 22 33 1490815444.0186 - E 12345678 [04] 00 11 22 33 1490815444.0200 B E 12345678 [04] 00 11 22 33 This example output was generated with the following small script: for i in 0 1 2 3 do cansend vcan0 12345678##${i}00112233 done Change-Id: I579f72383ebb92a979251bd41fe3a6fa58f33f04 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | canbusutil: Rename command line option for extra informationAndre Hartmann2017-03-301-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | At the moment, only CAN FD flags are shown, but this is very likely to be extended to TX/RX printing soon. Change-Id: Ibdf6014645fdc5a4d3791ab12ce298f4e2a4b3c6 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | Add plugin to test CanBusFactory compatibilityAndre Hartmann2017-03-307-2/+242
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add a plugin that implements the now deprecated QCanBusFactory to assure this still works. Change-Id: Idd0e121ebdd324bacc01c1805ab43749a85c7a84 Reviewed-by: Alex Blasche <alexander.blasche@qt.io> Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
* | | QCanBusFactory: Restore binary compatibility to Qt 5.8Andre Hartmann2017-03-3010-32/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a new QCanBusFactoryV2 to load plugins with the new virtual function availableDevices(). To be able to still load plugins compiled against Qt 5.8, both factories have to be considered when loading plugins and creating devices. Change-Id: I843cdd0b82193e391699daa298454c8b9522e860 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | QCanBus: Remove unused forward declarationAndre Hartmann2017-03-281-2/+0
| | | | | | | | | | | | | | | Change-Id: I608b890166deea40e20cc3790790479960aba6b2 Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | canbusutil: Add CAN FD flag handlingAndre Hartmann2017-03-275-8/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * For receiving, force the CAN plugin into CAN FD mode and show the CAN FD flags like the CAN Example * For sending, check if an odd number of data bytes is given and interpret the first data char as CAN FD flags Task-number: QTBUG-56464 Change-Id: I235827ea59d58744c34d5259e5637ad9c3326112 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | canbusutil: Unify CAN bus frame terminologyAndre Hartmann2017-03-271-2/+2
| | | | | | | | | | | | | | | | | | | | | Change-Id: I23fefe9f8db3656fccb08c3968d6174a91d82412 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Rolf Eike Beer <eb@emlix.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | VectorCAN: Update documentation for QCanBusDevice::ReceiveOwnKeyAndre Hartmann2017-03-271-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | Amends commit bb77953d24bd633eb87caf80df1fbaa3d8cf6da5 Change-Id: Idc52125a043294c0f9c5eedb9d4883373d924cb9 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Mihai Baneu <bdmihai@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | SocketCAN: Sort device info list by device nameAndre Hartmann2017-03-141-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise the sorting depends on the directory structure in /sys/class/net/ Change-Id: I95cc3d5f5d88f16c3511a9594eb4c466f807ef71 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | GenericBusPlugin: Remove unneded includeAndre Hartmann2017-03-131-2/+0
| | | | | | | | | | | | | | | Change-Id: I118d1a5e2ec4fff583476a1a643a37764ce2b49b Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | QCanBusDeviceInfo: cancel friendship to GenericBusPluginAndre Hartmann2017-03-134-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GenericBusPlugin is just used for internal testing. It does not need special handling, it can just behave like the other plugins. Change-Id: I55cb1133b51cb5f6ab3ca49484d3ba795867c2bb Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | QCanBusDeviceInfo: Improve documentationAndre Hartmann2017-03-121-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Amends the introducing commit 55d405697c79c4ca23535140e60915e89b4054e3 as well as the API improvement in commit 15d3834f6dbe319827abdbdb92383846dd7cac3c Change-Id: Ie1da751cf3cbe743bc2d159bb1515af6b3d23fe0 Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
* | | QCanBus: Sort includes and remove unneeded onesAndre Hartmann2017-03-071-3/+2
| | | | | | | | | | | | | | | | | | Change-Id: Ic47e936f6b6320e06da8d999cffad729e2f273b8 Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
* | | QCanBusDeviceInfo: Public API improvementAndre Hartmann2017-03-062-6/+15
| | | | | | | | | | | | | | | | | | | | | | | | Fixes issues pointed out in the API review. Change-Id: Ia335440e7df48ee93b3243cf5686e23a3859febc Reviewed-by: Marc Mutz <marc.mutz@kdab.com> Reviewed-by: Alex Blasche <alexander.blasche@qt.io>