>On Mon, Sep 23, 2024 at 05:48:28PM GMT, Hermes Wu wrote: >> From: Hermes Wu <Hermes.wu@xxxxxxxxxx> >> >> Changes in v3: >> -add detials about fail item and changes. >> >> >> Fix HDCP CTS fail items on UNIGRAF DRP-100 >> >> DUT must Support 127 devices. >> DUT must check BSTATUS when receive CP_IRQ. >> DUT must enable encryption when R0' is ready. >> DUT must retry V' check 3 times. >> it6505 must read DRP-100 KSV FIFO by FIFO mode. >> it6505 should restart HDCP within 5s if KSV not ready. > >Still not readable. > >English text, please. Split the patch to fix one issue at a time. >Describe the _reason_ for the change. Annotate fixes with Fixes tags. > with fixes tag include drm/bridge like this ? => "Fixes: drm/bridge: it6505: HDCP CTS fail 1B-xx" About the reason about bug fixes. for example, the 1B-01 device count. will this readable? " When connect to HDCP repeater, it6505 must support 127 downstream devices. " And this will be only one change in a patch? >> >> Signed-off-by: Hermes Wu <Hermes.wu@xxxxxxxxxx> >> --- >> drivers/gpu/drm/bridge/ite-it6505.c | 112 ++++++++++++++++++---------- >> 1 file changed, 74 insertions(+), 38 deletions(-) > >-- >With best wishes >Dmitry BR, Hermes