Messages in this thread | | | Date | Mon, 13 Jan 2014 09:22:45 +0000 | Subject | Re: [PATCH 3/3] gpio: MAX6650/6651 support | From | Laszlo Papp <> |
| |
>>> +#define PIN_NUMBER 5 >> >> As I can see this is really a GPIO+pin control driver it shall be >> moved to drivers/pinctrl. > > Hmm, but then I am not sure why the gpio-max*.c are similar in the > drivers/gpio area. Could you please elaborate? They are somewhat > similar to my understanding, but perhaps there is some fundamental > difference I am not aware of?
I was giving a second thought to this. Would it be acceptable to add the gpio driver now, and once the need arises, add the pinctrl thin layer on top of it? My concern is that I would not use anything else than the gpio functionality of these pins. It would be a needless additional work (i.e. investment) for my project and employer.
Perhaps, the layer on top of that can be added later without any drawback if anyone ever finds the need to have more functionality supported by these pins?
Shall look forward to hearing your opinion.
| |