| Message ID | 20201210104747.3416781-1-ikjn@chromium.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | Release allocated periodic bandwidth data from reset_bandwidth() | expand |
On Thu, 2020-12-10 at 18:47 +0800, Ikjoon Jang wrote: > xhci-mtk releases allocated TT bandwidth data only when whole > endpoints of a device are dropped as there're only {add|drop}_endpoint() > hooks are defined. This patchset adds more hooks and releases all > bandwidth data from reset_bandwidth() path, not drop_endpoint(). > > > Changes in v2: > - fix a 0-day warning from unused variable > - split one big patch into three patches > - bugfix in hw flags > > Ikjoon Jang (3): > usb: xhci-mtk: code cleanups in getting bandwidth table > usb: xhci-mtk: delay association of tt and ep > usb: xhci-mtk: fix unreleased bandwidth data > > drivers/usb/host/xhci-mtk-sch.c | 180 ++++++++++++++++++++------------ > drivers/usb/host/xhci-mtk.h | 13 +++ > drivers/usb/host/xhci.c | 9 ++ > 3 files changed, 133 insertions(+), 69 deletions(-) Thanks for your patch, I'll test it and check it with our DE >
On Fri, Dec 11, 2020 at 9:53 AM Chunfeng Yun <chunfeng.yun@mediatek.com> wrote: > > On Thu, 2020-12-10 at 18:47 +0800, Ikjoon Jang wrote: > > xhci-mtk releases allocated TT bandwidth data only when whole > > endpoints of a device are dropped as there're only {add|drop}_endpoint() > > hooks are defined. This patchset adds more hooks and releases all > > bandwidth data from reset_bandwidth() path, not drop_endpoint(). > > > > > > Changes in v2: > > - fix a 0-day warning from unused variable > > - split one big patch into three patches > > - bugfix in hw flags > > > > Ikjoon Jang (3): > > usb: xhci-mtk: code cleanups in getting bandwidth table > > usb: xhci-mtk: delay association of tt and ep > > usb: xhci-mtk: fix unreleased bandwidth data > > > > drivers/usb/host/xhci-mtk-sch.c | 180 ++++++++++++++++++++------------ > > drivers/usb/host/xhci-mtk.h | 13 +++ > > drivers/usb/host/xhci.c | 9 ++ > > 3 files changed, 133 insertions(+), 69 deletions(-) > Thanks for your patch, I'll test it and check it with our DE Thanks, I will upload v3. But I don't expect any logic changes from v2. Can you please give me feedback on v2 if you find anything? > > > >
On Fri, 2020-12-11 at 14:36 +0800, Ikjoon Jang wrote: > On Fri, Dec 11, 2020 at 9:53 AM Chunfeng Yun <chunfeng.yun@mediatek.com> wrote: > > > > On Thu, 2020-12-10 at 18:47 +0800, Ikjoon Jang wrote: > > > xhci-mtk releases allocated TT bandwidth data only when whole > > > endpoints of a device are dropped as there're only {add|drop}_endpoint() > > > hooks are defined. This patchset adds more hooks and releases all > > > bandwidth data from reset_bandwidth() path, not drop_endpoint(). > > > > > > > > > Changes in v2: > > > - fix a 0-day warning from unused variable > > > - split one big patch into three patches > > > - bugfix in hw flags > > > > > > Ikjoon Jang (3): > > > usb: xhci-mtk: code cleanups in getting bandwidth table > > > usb: xhci-mtk: delay association of tt and ep > > > usb: xhci-mtk: fix unreleased bandwidth data > > > > > > drivers/usb/host/xhci-mtk-sch.c | 180 ++++++++++++++++++++------------ > > > drivers/usb/host/xhci-mtk.h | 13 +++ > > > drivers/usb/host/xhci.c | 9 ++ > > > 3 files changed, 133 insertions(+), 69 deletions(-) > > Thanks for your patch, I'll test it and check it with our DE > > Thanks, I will upload v3. > But I don't expect any logic changes from v2. > Can you please give me feedback on v2 if you find anything? Ok > > > > > > > >
xhci-mtk releases allocated TT bandwidth data only when whole endpoints of a device are dropped as there're only {add|drop}_endpoint() hooks are defined. This patchset adds more hooks and releases all bandwidth data from reset_bandwidth() path, not drop_endpoint(). Changes in v2: - fix a 0-day warning from unused variable - split one big patch into three patches - bugfix in hw flags Ikjoon Jang (3): usb: xhci-mtk: code cleanups in getting bandwidth table usb: xhci-mtk: delay association of tt and ep usb: xhci-mtk: fix unreleased bandwidth data drivers/usb/host/xhci-mtk-sch.c | 180 ++++++++++++++++++++------------ drivers/usb/host/xhci-mtk.h | 13 +++ drivers/usb/host/xhci.c | 9 ++ 3 files changed, 133 insertions(+), 69 deletions(-)