lkml.org 
[lkml]   [2017]   [Jan]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v19 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer
    Hi,

    On Wed, Dec 21, 2016 at 02:45:48PM +0800, fu.wei@linaro.org wrote:
    > From: Fu Wei <fu.wei@linaro.org>
    >
    > This patchset:
    > (1)Preparation for adding GTDT support in arm_arch_timer:
    > 1. Move some enums and marcos to header file;
    > 2. Add a new enum for spi type;
    > 3. Improve printk relevant code;
    > 4. Rename some enums and defines;

    Can we please rework these first few patches into a series:

    1. Clean up printk() usage
    2. Rename the type macros
    3. Rename the PPI enum & enum values
    4. Move the type macro and PPI enum into a header
    5. Add new enum for SPIs

    Please leave CNTHCL_* as they originally were. It's not part of the
    interface of the driver, and leaving them as they are will save us an
    interdependency with KVM.

    I'm happy to queue that immediately, as it's purely structural and
    shouldn't have a functional impact anywhere. That will also shrink the
    series a bit, and we can rebase the following patches atop of that.

    > 5. Rework PPI determination;
    > 6. Rework counter frequency detection;
    > 7. Refactor arch_timer_needs_probing, move it into DT init call
    > 8. Introduce some new structs and refactor the MMIO timer init code
    > for reusing some common code.

    I'll review these in separate replies.

    Thanks,
    Mark.

    \
     
     \ /
      Last update: 2017-01-16 18:01    [W:2.205 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site