linux-kernel - Re: [PATCH V6 0/5] Add Genesys Logic GL975x support

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: <CACT4zj-43ShHd4YrUahJ+tza-b18AtB9TYgTk74HonXd7iKEpw@mail.gmail.com>
Date:   Fri, 30 Aug 2019 09:23:04 +0800
From:   Ben Chuang <benchuanggli@...il.com>
To:     Ulf Hansson <ulf.hansson@...aro.org>
Cc:     Adrian Hunter <adrian.hunter@...el.com>,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "Michael K. Johnson" <johnsonm@...lj.org>,
        Ben Chuang <ben.chuang@...esyslogic.com.tw>
Subject: Re: [PATCH V6 0/5] Add Genesys Logic GL975x support

On Thu, Aug 29, 2019 at 9:34 PM Ulf Hansson <ulf.hansson@...aro.org> wrote:
>
> On Tue, 27 Aug 2019 at 02:32, Ben Chuang <benchuanggli@...il.com> wrote:
> >
> > From: Ben Chuang <ben.chuang@...esyslogic.com.tw>
> >
> > The patches modify internal clock setup to match SD Host Controller
> > Simplified Specifications 4.20 and support Genesys Logic GL9750/GL9755 chipsets.
> >
> > V6:
> >  - export sdhci_abot_tuning() function symbol
> >  - use C-style comments
> >  - use BIT, FIELD_{GET,PREP} and GENMASK to define bit fields of register
> >  - use host->ops->platform_execute_tuning instead of mmc->ops->execute_tuning
> >  - call sdhci_reset() instead of duplicating the code in sdhci_gl9750_reset
> >  - remove .hw_reset
> >  - use condition define CONFIG_MMC_SDHCI_IO_ACCESSORS for read_l
>
> Applied patch 1-4, deferring patch5 for another version to fix Adrian's comment.
>
> Kind regards
> Uffe

Hi, Uffe,

So happy to get the good news.
 Thank you and Adrian for reviewing and helping.

Sincerely yours,
Ben Chuang