On Wed, Oct 28, 2015 at 12:52 PM, Pramod Kumar <pramo...@broadcom.com> wrote:
> I discussed with ASIC team regarding this iProc GPIO block. They use a > library to create the GPIOs block where "total number of GPIO pins( let say > N) in GPIO block" is used as an parameter. > Library uses a construct for *a* GPIO pin. This gets instantiated N times to > create a complete GPIO block with N pins. > > All iProc based SoCs uses this library. So I'm not sure whether attaching > "total number of GPIOs pins" to compatible-string make sense in this case. > I personally feel that passing this number from the DT makes more sense here. > Any iProc based future as well as current SoCs would be able to use this > driver without any change. > > Please advise us in this case. Hm! You make a good case. But this contradicts the traditional use of ngpios. But on the other hand: git grep ngpio Documentation/devicetree/bindings/gpio/ Gives at hand that the use of ngpio[s] is a complete mess. :( I will think about patching the standard bindings to fix this mess and include your case. Give me some time. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
