[U-Boot] [linux-sunxi] Re: [PATCH 3/6] sunxi: Add the R8 DTSI

Tom Rini
2015-09-18T15:49:06Z

On Fri, Sep 18, 2015 at 11:39:25AM -0400, Hans de Goede wrote:
> Hi,
>
> On 09/18/2015 11:39 AM, Tom Rini wrote:
> >On Fri, Sep 18, 2015 at 11:17:00AM -0400, Hans de Goede wrote:
> >>Hi,
> >>
> >>On 09/18/2015 11:02 AM, Tom Rini wrote:
> >>>On Fri, Sep 18, 2015 at 02:06:17PM +0200, Maxime Ripard wrote:
> >>>
> >>>>The R8 is very close to the A13, but it still has a few differences,
> >>>>notably a composite output, which the A13 lacks.
> >>>>
> >>>>Add a DTSI based on the A13's to hold those differences.
> >>>>
> >>>>Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> >>>
> >>>Other than needing to use SPDX tags instead:
> >>>Reviewed-by: Tom Rini <trini@konsulko.com>
> >>
> >>This is in essence a partial sync with the kernel tree wrt the dts
> >>files, so no SPDX tags.
> >
> >Really? I'd have sworn that we were doing that even on kernel files,
>
> I don't know. I'm fine either way, but I do not think we should be
> adding SPDX tags on the u-boot side only for these files, since those
> will just get overwritten (removed) on the next sync with the kernel.

Given the current (and it's going to need some work to acheive and then
keep) goal of just re-using kernel dts files, yeah, lets avoid noise
here. I think the other places we sync have enough other stuff going on
that it's not just a drop-in.

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150918/f1f2c4ce/attachment.sig>