linux-kernel - Re: [PATCH v1 1/2] usb: host: ehci-sunplus: Add driver for ehci in Sunplus SP7021

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 14 Feb 2022 17:48:20 +0800
From:   施錕鴻 <vincent.sunplus@...il.com>
To:     Greg KH <gregkh@...uxfoundation.org>
Cc:     stern@...land.harvard.edu, linux-kernel@...r.kernel.org,
        linux-usb@...r.kernel.org, robh+dt@...nel.org,
        devicetree@...r.kernel.org, wells.lu@...plus.com
Subject: Re: [PATCH v1 1/2] usb: host: ehci-sunplus: Add driver for ehci in
 Sunplus SP7021

Greg KH <gregkh@...uxfoundation.org> 於 2022年2月14日 週一 下午5:14寫道：
>
> On Mon, Feb 14, 2022 at 05:03:00PM +0800, 施錕鴻 wrote:
> > Hi, Greg
> >     About this issue, my colleague Hammer Hsieh has explained it to
> > you recently in the mail of "[PATCH v7 2/2] serial: sunplus-uart: Add
> > Sunplus
> > SoC UART Driver". The ehci driver and uart one are in the same Sunplus Soc.
>
> I do not know what you are referring to, sorry.  Remember we get
> thousands of emails a week.
>
> Please be explicit and make the code work properly for each patch you
> submit.
>
> thanks,
>
> greg k-h

Hi, Greg
    About data incoherence issue between memory bus and peripheral bus.
In case of AXI bus, use non-posted write can avoid data incoherence issue.
    What if in case of post write:
Send a specific command after last write command. SDCTRL identify specific
command, means previous write command done. Then send the interrupt
signal to interrupt controller. And then interrupt controller sends done signal
to Master. Master receives done signal, means write command done. Then
issue a interrupt or proceed next write command.
    Thanks for your review.