lkml.org 
[lkml]   [2016]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v5 0/5] Add support for legacy SCPI protocol
    From
    Date


    On 19/10/16 13:51, Neil Armstrong wrote:
    > This patchset aims to support the legacy SCPI firmware implementation that was
    > delivered as early technology preview for the JUNO platform.
    >
    > Finally a stable, maintained and public implementation for the SCPI protocol
    > has been upstreamed part of the JUNO support and it is the recommended way
    > of implementing SCP communication on ARMv8 platforms.
    >
    > The Amlogic GXBB platform is using this legacy protocol, as the RK3368 & RK3399
    > platforms. This patchset will only add support for Amlogic GXBB SoC.
    >
    > This patchset add support for the legacy protocol in the arm_scpi.c file,
    > avoiding code duplication.
    >
    > This patchset is rebased against scpi-updates/for-next from [2] and with
    > already merged patches [3], [4] and [5] and ommited in this patchset.
    >
    > Last RFC discution thread can be found at : https://lkml.org/lkml/2016/8/9/210
    >
    > Changes since v4 at : http://lkml.kernel.org/r/1475652814-30619-1-git-send-email-narmstrong@baylibre.com
    > - Removed legacy locking scheme
    > - Removed cmd copy back after token insert
    > - Various cleanups
    >
    > Changes since v3 at : http://lkml.kernel.org/r/1473262477-18045-1-git-send-email-narmstrong@baylibre.com
    > - Changed back author to Sudeep Holla for first patch
    > - Merged legacy functions to scpi_send_message, tx_prepare and handle_remote_message
    > - Added legacy locking scheme
    > - Merged back legacy_scpi_sensor_get_value into scpi_sensor_get_value
    > - Rebased on linux-next-20161004 with patchset [1]
    >
    > Changes since v2 at : http://lkml.kernel.org/r/1471952816-30877-1-git-send-email-narmstrong@baylibre.com
    > - Added command indirection table and use it in each commands
    > - Added bitmap for high priority commands
    > - Cleaned up legacy tx_prepare/handle_message to align to standard functions
    > - Dropped legacy_scpi_ops
    >
    > Changes since v1 at : http://lkml.kernel.org/r/1471515066-3626-1-git-send-email-narmstrong@baylibre.com
    > - Dropped vendor_send_message and rockchip vendor mechanism patches
    > - Merged alternate functions into main functions using is_legacy boolean
    > - Added DT match table to set is_legacy to true
    > - Kept alternate scpi_ops structure for legacy
    >
    > [1] http://lkml.kernel.org/r/1475595430-30075-1-git-send-email-narmstrong@baylibre.com
    > [2] git.kernel.org/sudeep.holla/linux
    > [3] scpi: Add cmd indirection table to prepare for legacy commands
    > [4] scpi: grow MAX_DVFS_OPPS to 16 entries
    > [5] dt-bindings: Add support for Amlogic GXBB SCPI Interface
    >
    > Neil Armstrong (5):
    > scpi: Add alternative legacy structures, functions and macros
    > scpi: Do not fail if get_capabilities is not implemented
    > scpi: Add support for Legacy match table for Amlogic GXBB SoC
    > ARM64: dts: meson-gxbb: Add SRAM node
    > ARM64: dts: meson-gxbb: Add SCPI with cpufreq & sensors Nodes
    >
    > arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 57 ++++++++
    > drivers/firmware/arm_scpi.c | 206 +++++++++++++++++++++++++---
    > 2 files changed, 245 insertions(+), 18 deletions(-)
    >

    Nice to see this diff stat from a whole new file legacy_scpi.c and 1000+
    delta. Thanks for working on this. I have applied the first 3 patches in
    this series with some subject/commit message changes to [1].

    I assume the DT changes needs to go via the corresponding platform
    maintainer.

    --
    Regards,
    Sudeep

    [1] git.kernel.org/sudeep.holla/linux/h/scpi-updates/for-next

    \
     
     \ /
      Last update: 2016-10-19 16:48    [W:3.090 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site