sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

device/xhci: Factor out common PORTSC code

Browse Source 
This commit factors out some code for XHCI port status values.

BUG=b:186792595
TEST=Built coreboot for volteer device

Change-Id: I045405ed224aa8f48f6f628b7d49ec6bafb450d7
Signed-off-by: Robert Zieba <robertzieba@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/67933
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Robert Zieba
2022-08-10 16:31:27 -06:00
committed by Martin L Roth
parent 065c5870e4
commit 4428195692
2 changed files with 42 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
src/include/device/xhci.h
View File

@@ -3,7 +3,7 @@
#ifndef __DEVICE_XHCI_H__
#define __DEVICE_XHCI_H__
#include <stdint.h>
#include <types.h>
#include <device/device.h>
#include <commonlib/bsd/cb_err.h>
@@ -12,6 +12,42 @@
#define XHCI_ECP_CAP_ID_LEGACY 1
#define XHCI_ECP_CAP_ID_SUPP 2
/* Status flags */
/* Wake on disconnect enable */
#define XHCI_STATUS_WDE BIT(26)
/* Wake on connect enable */
#define XHCI_STATUS_WCE BIT(25)
/* Port link status change */
#define XHCI_STATUS_PLC BIT(22)
/* Connect status change */
#define XHCI_STATUS_CSC BIT(17)
/* Port link status */
#define XHCI_STATUS_PLS_SHIFT 5
#define XHCI_STATUS_PLS_MASK (0xf << XHCI_STATUS_PLS_SHIFT)
#define XHCI_STATUS_PLS_RESUME (15 << XHCI_STATUS_PLS_SHIFT)
static inline bool xhci_portsc_csc(uint32_t port_status)
{
return port_status & XHCI_STATUS_CSC;
}
static inline bool xhci_portsc_wake_capable(uint32_t port_status)
{
return (port_status & XHCI_STATUS_WCE) |
(port_status & XHCI_STATUS_WDE);
}
static inline bool xhci_portsc_plc(uint32_t port_status)
{
return port_status & XHCI_STATUS_PLC;
}
static inline bool xhci_portsc_resume(uint32_t port_status)
{
return (port_status & XHCI_STATUS_PLS_MASK) == XHCI_STATUS_PLS_RESUME;
}
struct xhci_supported_protocol {
union {
uint32_t reg0;

43
src/soc/intel/common/block/xhci/elog.c
View File

@@ -2,45 +2,12 @@
#include <device/mmio.h>
#include <device/pci_ops.h>
#include <device/xhci.h>
#include <elog.h>
#include <intelblocks/xhci.h>
#include <soc/pci_devs.h>
#include <stdint.h>
/* Wake on disconnect enable */
#define XHCI_STATUS_WDE (1 << 26)
/* Wake on connect enable */
#define XHCI_STATUS_WCE (1 << 25)
/* Port link status change */
#define XHCI_STATUS_PLC (1 << 22)
/* Connect status change */
#define XHCI_STATUS_CSC (1 << 17)
/* Port link status */
#define XHCI_STATUS_PLS_SHIFT (5)
#define XHCI_STATUS_PLS_MASK (0xF << XHCI_STATUS_PLS_SHIFT)
#define XHCI_STATUS_PLS_RESUME (15 << XHCI_STATUS_PLS_SHIFT)
static bool xhci_csc_set(uint32_t port_status)
{
return !!(port_status & XHCI_STATUS_CSC);
}
static bool xhci_wake_capable(uint32_t port_status)
{
return !!((port_status & XHCI_STATUS_WCE) |
(port_status & XHCI_STATUS_WDE));
}
static bool xhci_plc_set(uint32_t port_status)
{
return !!(port_status & XHCI_STATUS_PLC);
}
static bool xhci_resume(uint32_t port_status)
{
return (port_status & XHCI_STATUS_PLS_MASK) == XHCI_STATUS_PLS_RESUME;
}
/*
* Check if a particular USB port caused wake by:
* 1. Change in connect/disconnect status (if enabled)
@@ -73,8 +40,8 @@ static bool xhci_port_wake_check(uintptr_t base, uint8_t num, uint8_t host_event
* connect/disconnect to identify if the port caused wake
* event for USB attach/detach.
*/
if (xhci_csc_set(port_status) &&
xhci_wake_capable(port_status)) {
if (xhci_portsc_csc(port_status) &&
xhci_portsc_wake_capable(port_status)) {
elog_add_event_wake(host_event, 0);
elog_add_event_wake(event, i + 1);
found = true;
@@ -85,8 +52,8 @@ static bool xhci_port_wake_check(uintptr_t base, uint8_t num, uint8_t host_event
* Check if PLC is set and PLS indicates resume to identify if
* the port caused wake event for USB activity.
*/
if (xhci_plc_set(port_status) &&
xhci_resume(port_status)) {
if (xhci_portsc_plc(port_status) &&
xhci_portsc_resume(port_status)) {
elog_add_event_wake(host_event, 0);
elog_add_event_wake(event, i + 1);
found = true;