Re: [PATCH] MIPS: rb532: move GPIOD definition into C-files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 29, 2022 at 4:46 AM Jackie Liu <liu.yun@xxxxxxxxx> wrote:
>
> From: Jackie Liu <liuyun01@xxxxxxxxxx>
>
> My kernel robot report build error from drivers/iio/adc/da9150-gpadc.c,
>
> We define GPIOD in rb.h, in fact he should only be used in gpio.c, but

he --> it

> it affects the driver da9150-gpadc.c which goes against the original
> intention of the design, just move it to his scope.

> At the same time, GPIONMIEN and IMASK6 are not used anywhere, just delete
> them.

While not being used it's good to leave (save) them since it might be
the only means of the HW documentation. I know that Git history will
keep that, but it is more explicit just to have them in the code.

...

>  #define DEV3TC         0x01003C
>  #define BTCS           0x010040
>  #define BTCOMPARE      0x010044
> -#define GPIOBASE       0x050000

...

> +#define GPIOBASE       0x050000

I think this one belongs to the header.

-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux