[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/20] sdcard: proper implementation of CRC
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 00/20] sdcard: proper implementation of CRC |
Date: |
Mon, 7 May 2018 17:09:48 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/04/2018 12:58 PM, Philippe Mathieu-Daudé wrote:
> This series emerged after last Coverity scan and Peter suggestion in:
> http://lists.nongnu.org/archive/html/qemu-devel/2018-04/msg05046.html
>
> (3) "proper" implementation of CRC, so that an sd controller
> can either (a) mark the SDRequest as "no CRC" and have
> sd_req_crc_validate() always pass, or (b) mark the SDRequest
> as having a CRC and have sd_req_crc_validate() actually
> do the check which it currently stubs out with "return 0"
>
> - Coverity issues fixed (up to patch 6)
> - crc16 now works
> - new functions documented
> - qtests added
>
> Regards,
>
> Phil.
>
> Philippe Mathieu-Daudé (20):
> sdcard: Use the ldst API
> sdcard: Extract sd_calc_frame48_crc7() from sd_req_crc_validate()
> sdcard: Rename the SDRequest as SDFrame48
> sdcard: Add sd_prepare_request[_with_crc]()
> sdcard: Use the sd_prepare_request*() functions
> sdcard: Add a "validate-crc" property
> sdcard: Constify sd_crc*()'s message argument
> sdcard: Fix sd_crc*() style
> sdcard: Expose sd_crc*() functions for QTest use
> sdcard: Expose sd_prepare_request*() functions for QTest use
> sdcard: Add test_sd_request_frame_crc7() qtest (request command CRC7)
> sdcard: Let sd_frame48_crc7_calc() work on response frames
> sdcard: Expose sd_prepare_frame48() for QTest use
> sdcard: Add test_sd_response_frame48_crc7 qtest (command response CRC7)
> sdcard: Add SDFrame136 struct and 136-bit SD response frames functions
> sdcard: Add test_sd_response_frame136_crc7() qtest
> sdcard: Add SDFrameData struct and data frame checksum functions
> sdcard: Fix sd_crc16()
> sdcard: Add test_sd_data_frame_crc16() qtest
> sdcard: Add test_sd_verify_cksum_frame48() qtest
Please disregard this series, I'll respin a simpler approach.
- [Qemu-devel] [PATCH 14/20] sdcard: Add test_sd_response_frame48_crc7 qtest (command response CRC7), (continued)
- [Qemu-devel] [PATCH 14/20] sdcard: Add test_sd_response_frame48_crc7 qtest (command response CRC7), Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 13/20] sdcard: Expose sd_prepare_frame48() for QTest use, Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 15/20] sdcard: Add SDFrame136 struct and 136-bit SD response frames functions, Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 16/20] sdcard: Add test_sd_response_frame136_crc7() qtest, Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 19/20] sdcard: Add test_sd_data_frame_crc16() qtest, Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 17/20] sdcard: Add SDFrameData struct and data frame checksum functions, Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 18/20] sdcard: Fix sd_crc16(), Philippe Mathieu-Daudé, 2018/05/04
- [Qemu-devel] [PATCH 20/20] sdcard: Add test_sd_verify_cksum_frame48() qtest, Philippe Mathieu-Daudé, 2018/05/04
- Re: [Qemu-devel] [PATCH 00/20] sdcard: proper implementation of CRC,
Philippe Mathieu-Daudé <=