Re: [PATCH 00/12] DRA7xx PRCM, hwmod and DT data

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

 



On Tue, 9 Jul 2013, Rajendra Nayak wrote:

> This series adds the data files for DRA7xx devices for
> PRCM, hwmod and DT. This is dependent on the core support [1]
> patches for DRA7xx and leaves out the clock data since its
> on its way to DT [2]
> 
> The regbit headers for prm and cm are cleaned up post the
> autogeneration as 95% was seen to be currently unused
> (prm-regbits-7xx.h was 100% unused and hence removed)
> 
> These patches along with the core support patches are available at:
> git://github.com/rrnayak/linux.git for-3.12/dra-core-data

Have glanced through these patches.  The form looks okay to me.  But again 
it's impossible to me to spot-check or verify almost anything in the data 
here, until some documentation is available.  Nor can I verify that the 
series works here, since I don't have a DRA7xx board.  It would be best 
for TI and for upstream Linux if us maintainers outside TI could at least 
test these patches, otherwise we are flying blind.

One minor comment is that I think the credit information in the comments 
of the data files needs to be qualified.  Instead of simply:

+ * Paul Walmsley (paul@xxxxxxxxx)
+ * Rajendra Nayak (rnayak@xxxxxx)
+ * Benoit Cousson (b-cousson@xxxxxx)

it would be better if it read something like:

* Generated by code originally written by:
* Paul Walmsley (paul@xxxxxxxxx)
* Rajendra Nayak (rnayak@xxxxxx)
* Benoit Cousson (b-cousson@xxxxxx)

or something similar.  Same for the other generated data files.

Are you fine with this change, or do you have some similar wording that 
you think is better?    No need to repost, I'll update the patches here.


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




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux