[meta-freescale] [meta-fsl-arm-extra][PATCH] 2/3] qmx6-addons-gpio: Add gpio.sh configuration script

[Otavio Salvador]

Hello Alex,

On Wed, Jan 28, 2015 at 10:10 AM, Alex de Cabo
<alejandro.de-cabo-garcia at congatec.com> wrote:
> This patch adds the gpio.sh script to the rootfs, and the
> needed links to make the listed GPIOs accesible via the
> /sys/class/gpio interface.
>
> The GPIOs [0..3] [5..7] are currently mapped as inputs, in
> order to define them as output, uncomment the
> "echo "out" ... direction" statement
> underneath the related pin export statement
>
> Pin mapping (Q7 GPIO vs. GPIO nr)
>
>         Q7, GPIO 0 <--> gpio130
>         Q7, GPIO 1 <--> gpio86 (disabled)
>         Q7, GPIO 2 <--> gpio122
>         Q7, GPIO 3 <--> gpio123
>         Q7, GPIO 5 <--> gpio111
>         Q7, GPIO 6 <--> gpio203
>         Q7, GPIO 7 <--> gpio110
>
> This patch needs the 4571a17 commit.
>
> Signed-off-by: Alex de Cabo <alejandro.de-cabo-garcia at congatec.com>

I understand this is for convenience but do you think it is worth
adding such a script inside the layer?

It seems for me this could be better if documented in the Congatec's
documentation or even in the Release Notes (adding Daiane on Cc so she
can comment if she thinks this applies or not).

The biggest problem I see with this kind of script is that it will
always be changed by the user and end not being tracked.

What do you think?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750