Message ID | 1369127450-15203-1-git-send-email-hdoyu@nvidia.com |
---|---|
State | New |
Headers | show |
On 05/21/2013 03:10 AM, Hiroshi Doyu wrote: > This patchset converts device tree files to use CLK defines. This > version was updated along with the latest DT changes. > > The following changes since commit 7479d8bb5650f43053d0e82fa44db668cf968f11: > > TESTING: Build Tegra USB as a module (2013-05-20 13:11:19 -0600) That commit is in my personal work-in-progress branch. This series should be based on Tegra's for-3.11/dt branch. This will make a difference (at least) in patch 6/6, where you've edited some DT nodes that don't yet exist upstream (i.e. Tegra's for-3.11/dt branch) and hence this series won't apply there. I assume you're planning to send a series for 3.11 that converts drivers/clk/tegra/ to use these new header files? If so, I'll need to apply patches 1, 3, 5 to a separate branch, so it can be pulled into the branch where the drivers/clk changes are applied, then apply patches 2, 4, 6 to Tegra's for-3.11/dt after merging in the branch with the header files.
On 05/21/2013 03:10 AM, Hiroshi Doyu wrote: > Use the Tegra20 CAR binding header (tegra20-car.h) to replace magic > numbers in the device tree. For example, > > - clocks = <&tegra_car 28>; > + clocks = <&tegra_car CLK_HOST1X>; > diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts > - clocks = <&tegra_car 112>, <&tegra_car 113>, <&tegra_car 94>; > + clocks = <&tegra_car TEGRA20_CLK_PLL_A>, <&tegra_car TEGRA20_CLK_PLL_A_OUT0>, <&tegra_car TEGRA20_CLK_CDEV1>; That's well over 80 columns. Can you wrap it and make sure there aren't any other instances of this? I guess checkpatch doesn't actually impose this rule on *.dts files, since it didn't complain about this.
On Tue, 21 May 2013 18:05:49 +0200 Stephen Warren <swarren@wwwdotorg.org> wrote: > On 05/21/2013 03:10 AM, Hiroshi Doyu wrote: > > This patchset converts device tree files to use CLK defines. This > > version was updated along with the latest DT changes. > > > > The following changes since commit 7479d8bb5650f43053d0e82fa44db668cf968f11: > > > > TESTING: Build Tegra USB as a module (2013-05-20 13:11:19 -0600) > > That commit is in my personal work-in-progress branch. This series > should be based on Tegra's for-3.11/dt branch. This will make a > difference (at least) in patch 6/6, where you've edited some DT nodes > that don't yet exist upstream (i.e. Tegra's for-3.11/dt branch) and > hence this series won't apply there. I'll rebase. > I assume you're planning to send a series for 3.11 that converts > drivers/clk/tegra/ to use these new header files? I think that I'll work on SMMU first, which diverted from the downstream now. So the above may come a bit later.
On 05/22/2013 12:12 AM, Hiroshi Doyu wrote: > On Tue, 21 May 2013 18:05:49 +0200 > Stephen Warren <swarren@wwwdotorg.org> wrote: > >> On 05/21/2013 03:10 AM, Hiroshi Doyu wrote: >>> This patchset converts device tree files to use CLK defines. This >>> version was updated along with the latest DT changes. >>> >>> The following changes since commit 7479d8bb5650f43053d0e82fa44db668cf968f11: >>> >>> TESTING: Build Tegra USB as a module (2013-05-20 13:11:19 -0600) >> >> That commit is in my personal work-in-progress branch. This series >> should be based on Tegra's for-3.11/dt branch. This will make a >> difference (at least) in patch 6/6, where you've edited some DT nodes >> that don't yet exist upstream (i.e. Tegra's for-3.11/dt branch) and >> hence this series won't apply there. > > I'll rebase. > >> I assume you're planning to send a series for 3.11 that converts >> drivers/clk/tegra/ to use these new header files? > > I think that I'll work on SMMU first, which diverted from the > downstream now. So the above may come a bit later. Really? SMMU support upstream is pretty low priority right now since we don't use it yet, and only converting one of the two potential users of this new binding header file means it isn't particularly useful.