>This needs to be named something intuitive like "vt8500_gpio_bank_regoffsets"
>Some kerneldoc intsead of the opaque comment above will also improve >readability a lot: >/** > * struct vt8500_gpio_bank_regoffsets > * @en: offset to enable register in the bank > * ... Changes made as requested. >My feeling however is that the "vt8500_chip->regoff" is wrong, which >would mean only the first bank works. The code adds the same offsets >per bank once more that it sets in this bank table. > Arnd Oh.. Thanks for picking that up Arnd. You are absolutely correct. I have removed the references for ->regoff. Left over from old code. Regards Tony Prisk _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss