Add an optional property to not retry multiple block read error
with several single block reads.
DooHyun Hwang (2):
dt-bindings: mmc: Add no-single-read-retry property
mmc: core: Add no single read retries
Documentation/devicetree/bindings/mmc/mmc-controller.yaml | 6 ++
drivers/mmc
read
operations as I/O error as much as retrying single block reads.
Signed-off-by: DooHyun Hwang
---
drivers/mmc/core/block.c | 3 ++-
drivers/mmc/core/host.c | 6 ++
include/linux/mmc/host.h | 3 +++
3 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/core/block.c b
Add an optional property to not retry multiple block read error
with several single block reads.
This property makes to handle read errors faster by not retrying
multiple block read errors with single block reads.
Signed-off-by: DooHyun Hwang
---
Documentation/devicetree/bindings/mmc/mmc
Add an optional property to not retry multiple block read error
with several single block reads.
DooHyun Hwang (2):
dt-bindings: mmc: Add no-single-read-retry property
mmc: core: Add no single read retries
Documentation/devicetree/bindings/mmc/mmc-controller.yaml | 6 ++
drivers/mmc
Add an optional property to not retry multiple block read error
with several single block reads.
This property makes to handle read errors faster by not retrying
multiple block read errors with single block reads.
Signed-off-by: DooHyun Hwang
---
Documentation/devicetree/bindings/mmc/mmc
read
operations as I/O error as much as retrying single block reads.
Signed-off-by: DooHyun Hwang
---
drivers/mmc/core/block.c | 3 ++-
drivers/mmc/core/host.c | 6 ++
include/linux/mmc/host.h | 3 +++
3 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/core/block.c b
Dear Maintainers and commiters.
Please review this patch for updating.
Thanks and regards.
DooHyun Hwang.
On 17/02/10 4:59 am, DooHyun Hwang wrote:
>Subject: [PATCH] mmc: core: add a power cycle when CMD11 fails
>
>A power cycle is required if CMD11 fails.
>CMD11 failure should be h
Card, CMD11 may fail but the card may have already
invoked the voltage switch sequence.
In this case, it is necessary to retry without voltage switching
after power cycle.
Signed-off-by: DooHyun Hwang
---
drivers/mmc/core/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
On 17/02/21 8:00 am, Adrian Hunter wrote:
>On 17/02/21 7:46 am, Adrian Hunter wrote:
>> On 17/02/21 7:22 am, DooHyun Hwang wrote:
>>> This makes to handle read errors faster by not retrying multiple
>>> block read(CMD18) errors with single block reads(CMD17).
>&g
Since the total error history count is unknown because the error history
records only the number of UFS_EVENT_HIST_LENGTH, add a member to count
each error history.
Signed-off-by: DooHyun Hwang
---
drivers/scsi/ufs/ufshcd.c | 3 +++
drivers/scsi/ufs/ufshcd.h | 1 +
2 files changed, 4 insertions
Hi
Thank you for your review and I found the below commit in linux-next.
So, I'll reject this patch.
>On 3/02/21 8:53 am, DooHyun Hwang wrote:
>> Since the total error history count is unknown because the error
>> history records only the number of UFS_EVENT_HIST_LENGTH, add
Since only print the recorded event history list,
add to print the counter value.
Signed-off-by: DooHyun Hwang
---
drivers/scsi/ufs/ufshcd.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index 721f55db181f..1ea920aeb701 100644
--- a
Remove unnecessary link startup command if it was completed.
UniPro stack is reset and enabled when ufshc is enabled.
The link startup command is issued after enabling ufshc,
if link startup is completed, there is no needed to issue again.
Signed-off-by: DooHyun Hwang
---
drivers/scsi/ufs
If the link startup command fails, trigger hardware reset to reset
the UFS device before link startup retry.
Signed-off-by: DooHyun Hwang
---
drivers/scsi/ufs/ufshcd.c | 8
1 file changed, 8 insertions(+)
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index
In the ufshcd init sequence, device state should be power-off
bacause the UFS device is not active.
Signed-off-by: DooHyun Hwang
---
drivers/scsi/ufs/ufshcd.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index
>>
>> Since only print the recorded event history list, add to print the
>> counter value.
>>
>> Signed-off-by: DooHyun Hwang
>Reviewed-by: Avri Altman
>
>Btw, You have the counter now in ufs-debugfs as well.
>
>Thanks,
>Avri
Thank you for yo
On Tue, 2 Mar 2021 at 10:38, Ulf Hansson wrote:
>On Wed, 10 Feb 2021 at 06:12, DooHyun Hwang
>wrote:
>>
>> A power cycle is required if CMD11 fails.
>> CMD11 failure should be handled as no response.
>>
>> If there is a timeout error that means no response to
On Tue, 2 Mar 2021 at 10:38, Ulf Hansson wrote:
>On Wed, 10 Feb 2021 at 06:12, DooHyun Hwang
>wrote:
>>
>> A power cycle is required if CMD11 fails.
>> CMD11 failure should be handled as no response.
>>
>> If there is a timeout error that means no respo
On Wed, 3 Mar 2021 at 12:00, Ulf Hansson wrote:
>
>On Wed, 3 Mar 2021 at 07:30, DooHyun Hwang wrote:
>>
>> On Tue, 2 Mar 2021 at 10:38, Ulf Hansson wrote:
>> >On Wed, 10 Feb 2021 at 06:12, DooHyun Hwang
>> >
>> >wrote:
>> >>
>> &
19 matches
Mail list logo