linux-kernel - Re: [PATCH v3 2/2] pinctrl: meson: rework pinmux ops

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: <CACRpkdbM94-ovZWN3quuXBrmPxB+O1C=dxAqPCspf-Bragg7Qw@mail.gmail.com>
Date:   Mon, 16 Oct 2017 14:01:50 +0200
From:   Linus Walleij <linus.walleij@...aro.org>
To:     Jerome Brunet <jbrunet@...libre.com>
Cc:     Kevin Hilman <khilman@...libre.com>,
        Carlo Caione <carlo@...one.org>,
        "linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
        "open list:ARM/Amlogic Meson..." <linux-amlogic@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 2/2] pinctrl: meson: rework pinmux ops

On Thu, Oct 12, 2017 at 2:40 PM, Jerome Brunet <jbrunet@...libre.com> wrote:

> This change prepare the introduction of new meson SoC. This new SoC will
> share the same gpio/pinconf registers but the pinmux part will be
> different. While the format of the data associated with each pinmux group
> will change, the way to handle pinmuxing will be similar.
>
> To deal with this new situation, the meson_pmx_struture is kept but the
> data associated to it is now generic. This allows to reuse the basic
> functions which would otherwise be copy/pasted in each pinmux driver
> (such as getting the name a count of groups and functions) Only the
> functions actually using this specific data is taken out of the common
> code and is handling the SoC pinmuxing
>
> Reviewed-by: Neil Armstrong <narmstrong@...libre.com>
> Reviewed-by: Kevin Hilman <khilman@...libre.com>
> Signed-off-by: Jerome Brunet <jbrunet@...libre.com>

Patch applied.

Yours,
Linus Walleij