diff options
author | Bjorn Andersson <bjorn.andersson@linaro.org> | 2017-11-28 17:14:12 -0800 |
---|---|---|
committer | Bjorn Andersson <bjorn.andersson@linaro.org> | 2017-11-28 17:40:24 -0800 |
commit | 34a47b4d2967532d7b9a3f6c27d37c7f17824a9d (patch) | |
tree | 4ed2bddcf61101d500f304269c6a8660729ddd63 | |
parent | c4852c48bf3ebf909b8b85f09877fc32221f71b5 (diff) |
firehose: Bump max payload size config parameter
The MaxPayloadSizeToTargetInBytes parameter of the configure command is
supposed to set the maximum size of the transfer unit. This size is
however limited by the use of the in-kernel qcserial driver and the use
of sector-sized writes. Yet increasing it, drastically bumps the
transfer speed, so leave it high.
NB. The max payload size supported by the device is part of the
configure response, if the device supports a value less than the
requested size - which we just ignore today.
Suggested-by: Kirill Kapranov <c_kkapra@qti.qualcomm.com>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
-rw-r--r-- | firehose.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -240,7 +240,7 @@ static int firehose_configure(int fd) node = xmlNewChild(root, NULL, (xmlChar*)"configure", NULL); xml_setpropf(node, "MemoryName", "ufs"); - xml_setpropf(node, "MaxPayloadSizeToTargetInBytes", "%d", 8192); + xml_setpropf(node, "MaxPayloadSizeToTargetInBytes", "%d", 1024 * 1024); xml_setpropf(node, "verbose", "%d", 0); xml_setpropf(node, "ZLPAwareHost", "%d", 0); |