[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdbUs1eWgZCvxNYqpnUox2=4nwewBi5YPqSBO8UDhikyLA@mail.gmail.com>
Date: Mon, 28 Oct 2024 14:02:49 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Inochi Amaoto <inochiama@...il.com>
Cc: Chen Wang <unicorn_wang@...look.com>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Inochi Amaoto <inochiama@...look.com>, Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, Guo Ren <guoren@...nel.org>,
Arnd Bergmann <arnd@...db.de>, Yixun Lan <dlan@...too.org>, linux-gpio@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-riscv@...ts.infradead.org
Subject: Re: [PATCH 2/3] pinctrl: sophgo: add support for SG2042 SoC
On Thu, Oct 24, 2024 at 8:44 AM Inochi Amaoto <inochiama@...il.com> wrote:
> Add base driver for SG2042 SoC and pin definition.
>
> Signed-off-by: Inochi Amaoto <inochiama@...il.com>
This driver is clearly duplicating a ton of code from pinctrl-cv18xx.c.
We have no interest in having to fix bugs in two places.
Please factor out the common code and use the same code
in the new driver if you can't make the new driver a simple
subdriver och pinctrl-cv18xx.c.
Yours,
Linus Walleij
Powered by blists - more mailing lists