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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox