From: Chris Ball <chris@printf.net>
To: Avi Shchislowski <Avi.Shchislowski@sandisk.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>,
"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
Alex Lemberg <Alex.Lemberg@sandisk.com>,
Yaniv Agman <Yaniv.Agman@sandisk.com>
Subject: Re: [PATCH 1/1 v2] mmc_utils: add ffu support
Date: Fri, 26 Feb 2016 14:45:05 +0000 [thread overview]
Message-ID: <86ziuncyz2.fsf@void.printf.net> (raw)
In-Reply-To: <SN1PR02MB13890B1700C4AE8F6FCD8C6C83A40@SN1PR02MB1389.namprd02.prod.outlook.com> (Avi Shchislowski's message of "Tue, 23 Feb 2016 11:27:44 +0000")
Hi Avi,
On Tue, Feb 23 2016, Avi Shchislowski wrote:
> Adding support for field firmware update over multiple command ioctl.
> As multiple command ioctl is supported only from kernel 4.4, this patch should be used against kernel 4.4 and above.
>
> Known issues:
> - There is no support for Multiple Block write commands (CMD25) in existing
> IOCTL implementation
> - In case MODE_OPERATION_CODES field is not supported by the device
> manual reset of the device/platform is required.
> The reset issue discussed in another email thread - " [RFC 0/6] mmc: Field Firmware Update"
>
> Change since v1:
> - modified copyright header
>
> Signed-off-by: Yaniv Agman <yaniv.agman@sandisk.com>
> Signed-off-by: Avi Shchislowski <avi.shchislowski@sandisk.com>
Thanks for resending. I'm getting some compile errors, any ideas?
mmc_cmds.c: In function ‘do_ffu’:
mmc_cmds.c:2076:31: error: invalid application of ‘sizeof’ to incomplete type ‘struct mmc_ioc_multi_cmd’
multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
^
mmc_cmds.c:2126:11: error: dereferencing pointer to incomplete type ‘struct mmc_ioc_multi_cmd’
multi_cmd->num_of_cmds = 3;
^
mmc_cmds.c:2162:23: error: ‘MMC_IOC_MULTI_CMD’ undeclared (first use in this function)
ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
^
- Chris.
--
Chris Ball <http://printf.net/>
next prev parent reply other threads:[~2016-02-26 14:52 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-23 11:27 [PATCH 1/1 v2] mmc_utils: add ffu support Avi Shchislowski
2016-02-26 14:45 ` Chris Ball [this message]
2016-02-26 14:50 ` Chris Ball
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86ziuncyz2.fsf@void.printf.net \
--to=chris@printf.net \
--cc=Alex.Lemberg@sandisk.com \
--cc=Avi.Shchislowski@sandisk.com \
--cc=Yaniv.Agman@sandisk.com \
--cc=linux-mmc@vger.kernel.org \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.