#6010 HIGH Update.: The EC SCI mask should be set in "sleep" mode.

[Zarro Boogs per Child]

#6010: The EC SCI mask should be set in "sleep" mode.
---------------------+------------------------------------------------------
  Reporter:  cjb     |       Owner:  dsaxena         
      Type:  defect  |      Status:  assigned        
  Priority:  high    |   Milestone:  Update.2 (8.2.0)
 Component:  kernel  |     Version:                  
Resolution:          |    Keywords:                  
  Verified:  0       |    Blocking:  5974            
 Blockedby:          |  
---------------------+------------------------------------------------------

Comment(by dsaxena):

 Replying to [comment:13 pgf]:
 > Replying to [comment:12 dsaxena]:
 >
 > > A sysfs one-file-per-bit interface guarantee an atomic update of the
 mask via a read/write lock.
 >
 > but how would you guarantee an atomic update of several bits at once?
 but perhaps that's not necessary in this case.

 I don't think it's completely needed. We have only one entity (OHM) that
 is going to be managing these bits.

 > >
 > > I've gone ahead and implemented the sysfs interface as this as it was
 fairly trivial; however, I'm wondering if in the long-run it may be better
 to not use sysfs and instead provide a full EC driver of some sort that
 exposes the EC interface to user space to do with it as it pleases.
 >
 > that needs to be balanced against possible future platform needs.  as
 has been said, sysfs does make future compatibility more likely, though at
 some cost.

 Agreed. There are also other issues with this as there maybe EC commands
 that we don't want userspace mucking with.

-- 
Ticket URL: <http://dev.laptop.org/ticket/6010#comment:14>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system