| From | Linus Walleij <> | Date | Tue, 2 Mar 2021 16:25:56 +0100 | Subject | Re: [PATCH 04/12] pinctrl: add a pincontrol driver for BCM6358 |
| |
On Thu, Feb 25, 2021 at 5:42 PM Álvaro Fernández Rojas <noltari@gmail.com> wrote:
> Add a pincotrol driver for BCM6358. BCM6358 allow overlaying different > functions onto the GPIO pins. It does not support configuring individual > pins but only whole groups. These groups may overlap, and still require > the directions to be set correctly in the GPIO register. In addition the > functions register controls other, not directly mux related functions. > > Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com> > Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
All of my review comments on the first driver in the series goes for this one as well, so copy/paste all my comments from there :D
The good thing is once you get it right in one driver you will get it right in all of them.
Consider bringing all common code together in a main driver shared by all these pin controllers as per the pattern of other pin controllers (se qcom for example).
Yours, Linus Walleij
|