Re: [GIT PULL 3/4] ARM: tegra: device tree changes for 3.12

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Stephen Warren <swarren@xxxxxxxxxxxxx> writes:

> On 08/20/2013 05:45 PM, Kevin Hilman wrote:
>> Stephen Warren <swarren@xxxxxxxxxxxxx> writes:
>> 
>>> This branch contains all *.dts (device tree) changes for Tegra.
>>> New features enabled are:
>>>
>>> * PMICs on Dalmore
>>> * CPU power-gating on Dalmore
>>> * HDMI output on Beaver
>>> * LP1 system suspend mode on almost all boards
>>> * PCIe support on numerous Tegra20/30 boards
>>> * USB support on Tegra30/114 boards
>>> * Audio capture on Beaver and Dalmore
>>> * Temperature sensor on Cardhu.
>>>
>>> ... along with a few DT cleanups.
>> 
>> Hmm, shouldn't I have a DT maintainer signoff on this stuff... oh,
>> wait... ;)
>
> One thought here on my DT binding maintainer role (hence CCing the other
> DT maintainers):
>
> I assume that the DT maintainers are reviewing the DT bindings, and not
> the *.dts files. There's certainly so much email defining/modifying
> bindings that I can't even keep up with that. So, I deliberately don't
> even attempt to look at anything other than the binding changes. Is this
> not what the role entails? If the role is supposed to include reviewing
> *.dts changes, then perhaps I should wait until someone else has
> reviewed such patches before applying them to the Tegra tree?

I was assuming that the DT maintainers would primarily focus on
bindings.  The platform maintainers should be primarily responsible for
the .dts files.

That being said, there will be no harm done if the DT reviewers have
some bandwidth to look at .dts files too. :)

> Of course, as Tegra maintainer I do actively look at the specific *.dts
> changes that go through my own tree.

Phew.

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux