In D50370#1160086, @imp wrote:

In D50370#1160085, @j_yoon.choi_samsung.com wrote:

In D50370#1159626, @j_yoon.choi_samsung.com wrote:

In D50370#1159535, @imp wrote:

I think this is ready to commit (or at least to start to try)... What do you think?

Yes, I think this is ready to commit!

I don't understand the commit process. Is there anything I need to do to commit?

Since you don't yet have commit permissions, I'll need to do that. I'll download it to my tree, make sure it compiles everywhere (or is disabled) and then push it into the repo. I just wanted to make sure that you didn't have anything else planned before I started that process...

In D50370#1159626, @j_yoon.choi_samsung.com wrote:

In D50370#1159535, @imp wrote:

I think this is ready to commit (or at least to start to try)... What do you think?

Yes, I think this is ready to commit!

In D50370#1159535, @imp wrote:

I think this is ready to commit (or at least to start to try)... What do you think?

Change UIC power mode ready time from 500ms to 2000ms

Apply review comments.

Apply review comments

Thank you for the kind review.
Your suggestions were very helpful in preparing this patch. :)

Apply review comment, Remove unnecessary functions

Rename functions

Fix a uic command bug

I measured the key performance metrics on the real device and confirmed that the results meet my goals.
For Write, if WriteBooster is applied, the bandwitch will increase.
Here are the results I measured on each device.

Fix a memory leak bug

Remove debug code

This patch is now ready for full review.
Thank you for your patience.

Apply clang-format

The UFSHCI driver now works on the Samsung Galaxy Book S and Lenovo Duet3.

Check the completion notification register when processing request completion

Number of bugs found during testing on Lenovo duet3

Fix a bug in interrupt enable register, Add Lenovo Duet3 11ian device id

Refactoring quirks

Move the quirk check routine to PCI attach.

The latest version of the UFS spec is paid, but you can download version 2.1 for free. Except for the MCQs, 4.1 and 2.1 are almost identical.
Please refer to them in your review.

Apply review comments

Thanks for the kind review! Most comments have been applied.
I'll mark the remaining ones as done after applying the review.
Thanks again!

Apply clang-format

In D50370#1151862, @emaste wrote:

We've updated style(9) to permit // comments but don't want styles mixed within a file/component. The //s here look like mostly temporary/debug code that won't be committed.

Comments starting with // may be used in both C and C++.  Styles should
not be mixed within a file or component.

There's also a file for clang-format which should produce acceptable results.

We can focus on architectural / high level approach review initially and do a style(9) pass once ready to commit.

In D50370#1151827, @jrtc27 wrote:

Obligatory style review:

• I see commented out code; either delete it or use #if(def) to gate it with an internal debug define
• FreeBSD uses /* C-style comments */ not // C++-style comments but I see a few instances of the latter
• Continuation lines aren't being consistently indented (I'm struggling to figure out what rule you've followed, even); they should be indented by four spaces relative to the block-level indent
• For combinations of bitwise operations etc where the operator precedence can be unintuitive we generally include redundant parentheses to avoid the need to think about whether it's correct without them (both for reading and writing)

You may find tools/build/checkstyle9.pl helpful, it can probably tell you about some of these, and maybe more I've not spotted. As an adaptation of QEMU's style checker it's not perfect though, so if it looks to be producing unexpected output then feel free to ignore those suggestions.

In D50370#1151769, @adrian wrote:

first pass - this on the surface looks like a well formatted / written driver. no magic numbers, consistent indenting, commented code, handling errors, thanks!

Add manual completion, Add comments to req_queue

Update UFS/UFSHCI version to 4.1, Add a Intel Lakefield quirk

Fix typo, update data_direction in UTRD, Add delay for HCE enable

I’d like to request an early review.
The items listed below are still incomplete, but I plan to finish them and update the patch while the review is in progress.