Re: [PATCH 1/5] dt-bindings: display: tegra: Add actmon information

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

 



>> An activity monitor (actmon) is used to measure the device runtime
>> utilization to help drive software power management policies.
>>
>> Extend the reg space to include actmon aperture for actmon configuration
>> through host1x.
>>
>> Extend the number of clocks to include actmon clock, which is shared
>> between unit actmons for different host1x clients.
>>
>> Signed-off-by: Johnny Liu <johnliu@xxxxxxxxxx>
>
> <form letter>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC. It might happen, that command when run on an older
> kernel, gives you outdated entries. Therefore please be sure you base
> your patches on recent Linux kernel.

Thanks for your correction. I will resend the patch and add necessary
people in the to-list, and add mailing-list to cc-list.

> Tools like b4 or scripts/get_maintainer.pl provide you proper list of
> people, so fix your workflow. Tools might also fail if you work on some
> ancient tree (don't, instead use mainline) or work on fork of kernel
> (don't, instead use mainline). Just use b4 and everything should be
> fine, although remember about `b4 prep --auto-to-cc` if you added new
> patches to the patchset.

Thanks for your guidance.

> You missed at least devicetree list (maybe more), so this won't be
> tested by automated tooling. Performing review on untested code might be
> a waste of time.

Thanks for your information. I will cover devicetree list in the next
patch series.

> Please kindly resend and include all necessary To/Cc entries.
> </form letter>

Ack.


Best regards,
Johnny



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux