在 2015/7/8 18:44, Thomas Gleixner 写道: > On Wed, 8 Jul 2015, majun (F) wrote: >> 在 2015/7/6 20:33, Thomas Gleixner 写道: >>> Care to explain what this does? It seems for some nodes you cannot >>> write the msi message. So how is that supposed to work? How is that >>> interrupt controlled (mask/unmask ...) ? >>> >> This function is used to write irq event id into vector register.Depends on >> hardware design, write operation is permitted in some mbigen >> node(nid=0,5,and >7), >> For other mbigen node, this register is read only. >> >> But only vector register has this problem. Other registers are ok for >> read/write. > > You still fail to explain how that works if the register is not > writeable. And the code wants a proper comment explaining it. >
Actually, the interrupt trigger type, device id and event id already encoded in mbigen chip. Depends on hardweare design, the device id and event id value in some mbigen nodes can't be modified, but some nodes can. For the mbigen node which event id can't be modified, we can use the default event id value (encoded in mbigen register). If the event id can be programmed, we can use this function to modify the event id value. [....] > > Aside of that, please look at the per-device MSI series Marc posted > (you were cc'ed). This is going to be where we are heading and your > driver should be based on that. Ok, i will change mbigen code based on Marc's patch. > > Thanks, > > tglx > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
