RE: [PATCH V3 0/2] AMD NTB V3 changes

From	"Allen Hubbe" <Allen.Hubbe@emc.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH V3 0/2] AMD NTB V3 changes
Date	2016-01-14 17:00 +0100
Message-ID	<qQStA-GV-15@gated-at.bofh.it> (permalink)
References	<qQStA-GV-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> From: Xiangliang Yu <Xiangliang.Yu@amd.com>
> 
> Main changes in V3
> 1. Remove all power managment related code.

Can you send the third patch with the power management code?  I think it is important functionality.

Furthermore, a review of the power management patch will help with getting the flush patch accepted.  If the power management patch is reviewed and acked as is, the flush patch is probably ok, too, as is.  If there are changes to power management that are also applicable to the flush patch, it would be good to keep it all consistent when adding the functionality to the api.

> 2. Put makefile and kconfig changes in first patch.
> 3. Call readl/writel directly, not use macros to hide it.

I think this is almost ready.  See comments in NTB: Add AMD PCI-Express NTB driver.

Allen

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

RE: [PATCH V3 0/2] AMD NTB V3 changes "Allen Hubbe" <Allen.Hubbe@emc.com> - 2016-01-14 17:00 +0100
  RE: [PATCH V3 0/2] AMD NTB V3 changes "Yu, Xiangliang" <Xiangliang.Yu@amd.com> - 2016-01-15 03:30 +0100

csiph-web