← Back to team overview

kernel-packages team mailing list archive

[Bug 1494735] Re: LightPath Dignostics framework on PowerNV platform

 

** Package changed: ubuntu => linux (Ubuntu)

** Changed in: linux (Ubuntu)
   Importance: Undecided => High

** Changed in: linux (Ubuntu)
       Status: New => Triaged

** Changed in: linux (Ubuntu)
     Assignee: Taco Screen team (taco-screen-team) => Canonical Kernel Team (canonical-kernel-team)

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1494735

Title:
  LightPath Dignostics framework on PowerNV platform

Status in linux package in Ubuntu:
  Triaged

Bug description:
  == Comment: #0 - VASANT HEGDE <hegdevasant@xxxxxxxxxx> - 2015-09-11 05:56:57 ==
  IBM x, i and p systems have Service indicators (LED) that help identify components (Guiding Light) and also to indicate a component in error (Light Path). Currently, Linux only has a couple of commands that cater to LightPath services, but not close to what is expected. The work involved is to deliver a LightPath framework that will help customer to identify a hardware component in error.

  commit 8a8d91817aec6b9ca7a289891f9e6f721100ec09
  Author: Anshuman Khandual <khandual@xxxxxxxxxxxxxxxxxx>
  Date:   Wed Aug 19 22:19:52 2015 +0530

      powerpc/powernv: Add OPAL interfaces for accessing and modifying system LED states
      
      This patch registers the following two new OPAL interfaces calls
      for the platform LED subsystem. With the help of these new OPAL calls,
      the kernel will be able to get or set the state of various individual
      LEDs on the system at any given location code which is passed through
      the LED specific device tree nodes.
      
          (1) OPAL_LEDS_GET_INDICATOR     opal_leds_get_ind
          (2) OPAL_LEDS_SET_INDICATOR     opal_leds_set_ind
      
      Signed-off-by: Anshuman Khandual <khandual@xxxxxxxxxxxxxxxxxx>
      Signed-off-by: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
      Acked-by: Stewart Smith <stewart@xxxxxxxxxxxxxxxxxx>
      Tested-by: Stewart Smith <stewart@xxxxxxxxxxxxxxxxxx>


  commit c159b5968ec9d3cbf16802189b29eb0ec17ae4d8
  Author: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
  Date:   Wed Aug 19 22:19:53 2015 +0530

      powerpc/powernv: Create LED platform device
      
      This patch adds platform devices for leds. Also export LED related
      OPAL API's so that led driver can use these APIs.
      
      Signed-off-by: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
      Acked-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
      Signed-off-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx>

  commit 84ad6e5cd3e8b365c893f31787864cae5500610b
  Author: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
  Date:   Wed Aug 19 22:19:54 2015 +0530

      leds/powernv: Add driver for PowerNV platform
      
      This patch implements LED driver for PowerNV platform using the existing
      generic LED class framework.
      


  commit 5d53be7d8c7ccf8eec1ce66c6b3573c01d16b755
  Author: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
  Date:   Sat Aug 22 09:33:55 2015 +1000

      powerpc/powernv: Fix mis-merge of OPAL support for LEDS driver
      
      When I merged the OPAL support for the powernv LEDS driver I missed a
      hunk.
      
      This is slightly modified from the original patch, as the original added
      code to opal-api.h which is not in the skiboot version, which is
      discouraged.
      
      Instead those values are moved into the driver, which is the only place
      they are used.

  
  commit d690740f22f6520873f96e66aae7119ec2215755
  Author: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
  Date:   Thu Aug 27 11:34:10 2015 +0530

      powerpc/powernv: Enable LEDS support
      
      Commit 84ad6e5c added LEDS support for PowerNV platform. Lets
      update ppc64_defconfig to pick LEDS driver.
      
      PowerNV LEDS driver looks for "/ibm,opal/leds" node in device
      tree and loads if this node exists. Hence added it as 'm'.
      
      Also note that powernv LEDS driver needs NEW_LEDS and LEDS_CLASS
      as well. Hence added them to config file.
      
      mpe: Also add them to pseries_defconfig, which is currently also used
      for powernv systems.
      
      Suggested-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
      Signed-off-by: Vasant Hegde <hegdevasant@xxxxxxxxxxxxxxxxxx>
      Cc: Stewart Smith <stewart@xxxxxxxxxxxxxxxxxx>
      Signed-off-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx>

  
  -Vasant

  == Comment: #1 - VASANT HEGDE <hegdevasant@xxxxxxxxxx> - 2015-09-11 05:58:20 ==
  All these patches went into current merge window (4.3).. Let me know if you want me to backport those patches.

  -Vasant

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1494735/+subscriptions