On 13 November 2014 17:28, Avi Shchislowski
<[email protected]> wrote:
>  The Field Firmware Update (FFU) feature is new for eMMC 5.0 spec (Jedec:
>  JESD84-B50.pdf)
>
>  
> http://www.jedec.org/standards-documents/technology-focus-areas/flash-memory-ssds-ufs-emmc/e-mmc
>
>
>
>  * This solution allows to:
>   - Complete eMMC 5.0 FFU procedure as an atomic operation, without being 
> interrupted by other IO requests
>   - Not limited Firmware data size. Using Multiple Write operations.
>   - Support of both EXT_CSD_MODE_OPERATION_CODES modes
>  * The solution is using "udev" device manager to transfer FW data from user 
> space to eMMC driver
>  * Pre-existing functions from mmc_test were used in this solution.
>
>  *New ioctl has been add:
>   -transfer the new Firmware data from user space
>    to the eMMC deivce and install the newfirmware.
>
> Signed-off-by: Avi Shchislowski <[email protected]>
> Signed-off-by: Alex Lemberg <[email protected]>

Guys, this isn't working. I am unable to review your patches!

1. I don't know what the patches do, because the commit
messages/headers are really poor.
2. Patches are too big.
3. You are mixing logical changes into one patch. Like when you move
code and change code in one patch. That's two different things and
thus should be kept separate.

Please go back and re-work this thoroughly before you decide to send a
new version.

Moreover I also think you need to read
"Documentation/SubmittingPatches", that should give you some more
hints on how to improve and help me as a maintainer.

Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to