From: Mike Rapoport <mike@compulab.co.il>
To: Tomi Valkeinen <tomi.valkeinen@nokia.com>
Cc: "ext Hiremath, Vaibhav" <hvaibhav@ti.com>,
Tony Lindgren <tony@atomide.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: Moving board patches from DSS2 to linux-omap
Date: Thu, 11 Feb 2010 13:50:37 +0200 [thread overview]
Message-ID: <4B73EF0D.8080904@compulab.co.il> (raw)
In-Reply-To: <1265884185.2433.239.camel@tubuntu.research.nokia.com>
Tomi Valkeinen wrote:
> Hi,
>
> As discussed previously, board file changes in DSS2 tree cause conflicts
> with linux-omap easily. There are currently three board file patches in
> DSS2's for-next branch:
>
> 722a97e4594b2041bbf18d95a913ba6dfaca87f2 omap3: cm-t35: add DSS2 display support
> 7a56267e775e469c64521179ccc958c8bb661dbf OMAP: AM3517: Enable DSS2 for AM3517EVM board
> 40e4e67c6dabcb9897b6823cce2297d6c3e78bbd OMAP: Enable DSS2 for OMAP3EVM board
>
> The problem here is of course that DSS2 tree may contain unmerged panel
> drivers, and those board file changes try to use these new panel
> drivers.
Well, the panel drivers are referenced by name in the board files, so merging board
file changes through linux-omap tree should not create merge conflicts and compile
problems. There just won't be display until DSS2 tree is merged. Or am I missing
something?
> Now, I don't think there's a perfect solution for this, but I think a
> working solution would be to put all board file changes to linux-omap
> tree (with the exception of some rare changes that don't compile without
> new DSS2 patches), but leave the kernel config unchanged.
>
> This way the board file contains references to new panel drivers, but as
> DSS2 nor the panel drivers are enabled in the Kconfig, the board file
> code is not used and everything should work as before.
>
> Then either I can have Kconfig patches in my tree, which are less likely
> to conflict, or the Kconfig changes can be done after both linux-omap
> and dss2 patches have been merged.
>
> How does this sound?
>
> Tomi
>
> Ps. I haven't actually tried those board file patches on top of
> linux-omap, but I don't see anything there that would cause compilation
> to fail.
>
>
--
Sincerely yours,
Mike.
next prev parent reply other threads:[~2010-02-11 11:51 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-11 10:29 Moving board patches from DSS2 to linux-omap Tomi Valkeinen
2010-02-11 11:50 ` Mike Rapoport [this message]
2010-02-11 12:04 ` Tomi Valkeinen
2010-02-11 18:52 ` Tony Lindgren
2010-02-12 10:44 ` Tomi Valkeinen
2010-02-12 20:42 ` Tony Lindgren
2010-02-13 10:34 ` Hiremath, Vaibhav
2010-02-13 10:43 ` Hiremath, Vaibhav
2010-02-15 7:43 ` Tomi Valkeinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B73EF0D.8080904@compulab.co.il \
--to=mike@compulab.co.il \
--cc=hvaibhav@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=tomi.valkeinen@nokia.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.