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] [day] [month] [year] [list]
Message-ID: <CAPDyKFqB4nxv5andQktNDjGkf-vPfbiFVwkZ7PGOWW9NbdM+qA@mail.gmail.com>
Date:   Fri, 6 Aug 2021 12:11:43 +0200
From:   Ulf Hansson <ulf.hansson@...aro.org>
To:     Lai Jason <jasonlai.genesyslogic@...il.com>
Cc:     Adrian Hunter <adrian.hunter@...el.com>,
        linux-mmc <linux-mmc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Ben Chuang <ben.chuang@...esyslogic.com.tw>,
        GregTu[杜啟軒] <greg.tu@...esyslogic.com.tw>,
        AKASHI Takahiro <takahiro.akashi@...aro.org>,
        Jason Lai <jason.lai@...esyslogic.com.tw>
Subject: Re: [RFC PATCH v3.2 00/29] Add support UHS-II for GL9755

On Fri, 6 Aug 2021 at 08:37, Lai Jason <jasonlai.genesyslogic@...il.com> wrote:
>
> On Tue, 03 Aug 2021 at 12:23,   Ulf Hansson <ulf.hansson@...aro.org>  wrote:
> >
> > On Thu, 22 Jul 2021 at 07:14, Jason Lai <jasonlai.genesyslogic@...il.com> wrote:
> > >
> > > Summary
> > > =======
> > > In these patches we focus on uhs2 side to address Ulf's suggestion which
> > > was mentioned in patch [mmc: core: Preparations to support SD UHS-II cards].
> > >
> > >
> > > I moved codes in uhs2.c to proper functions in sd_uhs2.c, which should meet
> > > Ulf's expection.
> > >
> > > Please review this series (particularly Patch 29/29).
> >
> > I think you have misunderstood the intent with the series that I
> > posted [1] a while ago. The idea was to add the foundation of what
> > should be needed in the mmc core to support UHS-II cards. Note, it's
> > only the foundation, not the complete code.
> >
> > In other words, I think you should base the next version of $subject
> > series *on top* of my series [1], rather than the opposite.
> >
> Actually, I understand your intention is to implement functions which
> can be applied to control all UHS-II cards.
> However, why do I choose to base it on “[RFC,v3.1,00/27] Add support
> UHS-II for GL9755”?
> Because it contains a complete UHS-II interface(both host & card side)
> and UHS-II card can work normally.

It doesn't matter if things "work".

The code/patches need to be better organized than currently,
especially the changes for the mmc core, which is why I stepped in and
tried to help out earlier. Otherwise it is impossible for me to review
the changes, but also to maintain the code in the long run.

The sdhci part is likely a bit more mature as it has been reviewed and
iterated with Adrian/Ben/Akashi-san. However, new/more changes needed
when my suggested changes for the mmc core come into play.

[...]

Kind regards
Uffe

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ