[PATCH 01/10] pinctrl: mediatek: data struct optimize and remove unused member

Linus Walleij linus.walleij at linaro.org
Tue May 19 01:49:19 PDT 2015


On Thu, May 14, 2015 at 3:35 AM, Hongzhou Yang
<hongzhou.yang at mediatek.com> wrote:

> From: Yingjoe Chen <yingjoe.chen at mediatek.com>
>
> struct mtk_desc_pin.chip, mtk_pinctrl_devdata.invser_offset
> and mtk_pinctrl_devdata.chip_type are never used in code.
> Remove them.
>
> Some per-pin data are using int for pin number and offsets.
> Change to short and rearrange to reduce const data size.
>
> Signed-off-by: Yingjoe Chen <yingjoe.chen at mediatek.com>
> Signed-off-by: Hongzhou Yang <hongzhou.yang at mediatek.com>

Patch applied.

However I suggest you use:

unsigned int => u32
unsigned short => u16
unsigned char => u8

These kernel-specific types are more to the point,
you immediately see how many bits they use, so a
patch chaning all of the types to u32/u16/u8 would
be welcome.

Yours,
Linus Walleij



More information about the Linux-mediatek mailing list