[0/2] pinctrl: ingenic: Cleanup & add lcd-8bit group

Hi Linus,

A cleanup patch to get rid of a lot of const data, and a patch to add
the lcd-8bit group for JZ4770 SoCs.

Based on your latest for-next branch, since there are already some
commits on pinctrl-ingenic.c there that aren't on v5.10-rc1.

Cheers,
-Paul

Paul Cercueil (2):
  pinctrl: ingenic: Get rid of repetitive data
  pinctrl: ingenic: Add lcd-8bit group for JZ4770

 drivers/pinctrl/pinctrl-ingenic.c | 1267 +++++++++++------------------
 1 file changed, 455 insertions(+), 812 deletions(-)

Hello,

On 2020/11/1 下午5:01, Paul Cercueil wrote:
> Hi Linus,
>
> A cleanup patch to get rid of a lot of const data, and a patch to add
> the lcd-8bit group for JZ4770 SoCs.
>
> Based on your latest for-next branch, since there are already some
> commits on pinctrl-ingenic.c there that aren't on v5.10-rc1.
>
> Cheers,
> -Paul
>
> Paul Cercueil (2):
>    pinctrl: ingenic: Get rid of repetitive data
>    pinctrl: ingenic: Add lcd-8bit group for JZ4770
>
>   drivers/pinctrl/pinctrl-ingenic.c | 1267 +++++++++++------------------
>   1 file changed, 455 insertions(+), 812 deletions(-)
>
Works fine on JZ4770, JZ4780, X1000, and X1830.

Tested-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>


Thanks and best regards!

On Sun, Nov 1, 2020 at 10:01 AM Paul Cercueil <paul@crapouillou.net> wrote:

> Abuse the pin function pointer to store the pin function value directly,

> when all the pins of a group have the same function value. Now when the

> pointer value is <= 3 (unsigned), the pointer value is used as the pin

> function; otherwise it is used as a regular pointer.

>

> This drastically reduces the number of pin function tables needed, and

> drops .data usage by about 2 KiB. Additionally, the few pin function

> tables that are still around now contain u8 instead of int, since the

> largest number that will be stored is 3.

>

> Signed-off-by: Paul Cercueil <paul@crapouillou.net>


Both patches applied.

Yours,
Linus Walleij