Re: [meta-fsl-arm][PATCH] linux-imx (3.0.35): Add Boundary Devices changes for imx6qsabrelite

[Gary Thomas <samoht.yrag@gmail.com>]

On 2012-11-30 05:44, Otavio Salvador wrote:
>
>
>
> On Fri, Nov 30, 2012 at 10:16 AM, Gary Thomas <samoht.yrag@gmail.com <mailto:samoht.yrag@gmail.com>> wrote:
>
>     On 2012-11-29 08:48, Otavio Salvador wrote:
>
>         We import the changes done in Bondary Devices tree and the defconfig
>         file provided by them.
>
>         Change-Id: I63e2d7560735586581fd13d5df60b__3d5e90d73a3
>         Signed-off-by: Otavio Salvador <otavio@ossystems.com.br <mailto:otavio@ossystems.com.br>>
>
>
>     This does make my touch screen work, thanks.  My camera module (OV5640 based) does
>     not work though, but I've not researched it much yet.
>
>     A couple of comments:
>     * Could you put the changes under mx6q instead of imx6qsabrelite, much like the other
>        files/changes that are present?  I ask because I have a derivative platform which has
>        a different name (just sabrelite - I got tired of mistyping imx6qsabrelite) and the
>        patch was not applied until I moved it about.
>
>
> It cannot; we need to apply it to the specific machine only and not all mx6q machines.

I didn't see anything in the patch that would break other imx6q devices, plus the way you have it
won't work if you wanted to also handle their newer nitrogen boards.

No problem - I can use OVERRIDES to accomplish the same thing.  I just add this to my derivative sabrelite.conf:
   OVERRIDES_append = ":imx6qsabrelite"

>
>     * More changes are necessary to make this useful for the target "out of the box", in particular:
>        - U-Boot needs to pass additional command line parameters to make the LCD panel work properly
>
>
> It is better to document it ... I am not sure if we ought to default to enable it or not for sabrelite. What do you think?

I'll propose some patches once I work out the details.  I'll make them target specific.

>
>        - I use the micro-SD slot as does the BoundaryDevices setup, but U-Boot only wants to boot
>          from the full size SD card
>
>
> We can change it; if you can, please send the patch for us and we apply it on U-Boot branch (https://github.com/Freescale/u-boot-imx/tree/patches-2012.07)

Same with this.

>
>        - The touch screen needs some additional files/tuning for X to work correctly (the X axis
>          is backwards, etc)
>
>
> If it uses evdev, we might try the newer xinput-callibrator that Mario (a coworker) has send to meta-oe mailing list.

It does use evdev.  I have a working config file which I'll also send a patch for.

>
>        - I'll investigate the camera soon (not top on my list yet)
>
>
> Good.
>
>        - The recent changes, in particular xf86-video-imxfb-vivante, do not build under Yocto/master
>          which has moved to Xorg server version 1.13.
>
>
> We're still working for danny so not yet in master; please stay in danny as this is the platform we're using for daily work for now.
>
>        - Why is GCC not tuned for the Cortex-A9 (currently armv7a is used)?  I rebuilt using
>              DEFAULTTUNE = "cortexa9hf-neon"
>          which works fine and is more in line with other Cortex-A9 platforms, e.g. PandaBoard.
>
>
> hardfloat can be a problem for ABI in some applications.

It's what Ubuntu (and I think Linaro) are using now.  Again, something I can handle in my derivative.

>
>     I'll see about proposing patches for all of these soon
>
>
> Very good; please do.
>
> But before looking at patches it is better for us to settle in what ought to be done for defaults...