Re: [ptxdist] [PATCH v2 1/2] dtc: Seperate path and filename of dts

[Alexander Aring <alex.aring@gmail.com>]

Hi,

On Wed, Jun 11, 2014 at 08:29:55AM +0200, Michael Olbrich wrote:
> Hi,
> 
> I've rewritten the patch taking these issues into account.
> I'll merge it after the release.
> 
> On Fri, May 16, 2014 at 08:45:19AM +0200, Alexander Aring wrote:
> > I detected a new issue with this patch.
> > 
> > 
> > First:
> > 
> > Before it was possible to have something like:
> > 
> > PTXCONF_DTC_OFTREE_DTS="/bar/foo/foobar.dts /foo/bar/barfoo.dts"
> > 
> > These files have a different basename. With your patch the basename must
> > be one unique basename. I mean okay, we could drop this and force the
> > ptxdist users that all dts files need to be under one location.
> > 
> > or
> > 
> > we make a for-each to search all directories of DTS_PATH, then we could
> > configure something like that:
> > 
> > DTC_OFTREE_DTS_PATHS="/bar/foo /foo/bar"
> > DTC_OFTREE_DTS="foobar.dts barfoo.dts"
> 
> I've used ':' as a separater for the path.
> 

just to clarify, what's the priority in this list?

Should we do that like PATH? Then I would prefer

${KERNEL_DIR}/arch/${PTXCONF_KERNEL_ARCH_STRING}/boot/dts:${PTXDIST_PLATFORMCONFIGDIR}/dts

instead

${PTXDIST_PLATFORMCONFIGDIR}/dts:${KERNEL_DIR}/arch/${PTXCONF_KERNEL_ARCH_STRING}/boot/dts

as default value. Then I can just copy the mainline dts file to ${PTXDIST_PLATFORMCONFIGDIR}/dts
without renaming and make some platform changes.

- Alex

-- 
ptxdist mailing list
ptxdist@pengutronix.de