Re: [PATCH v2 0/4] Style fixes: open code obfuscating macros

[Jon Mason]

On Thu, Jan 26, 2017 at 3:00 PM, Logan Gunthorpe <logang@xxxxxxxxxxxx> wrote:
> Hi,
>
> It's been a couple weeks... Any thoughts on this?

My apologies for not responding sooner.  A large rework of the core
NTB code was done prior to your patches (for IDT NTB support).
Unfortunately, after those changes this series does not apply cleanly.
I understand the changes being made, and will fix-up the patches to
make it apply cleanly on the updated code base.

Thanks,
Jon

>
> Thanks,
>
> Logan
>
> On 10/01/17 05:33 PM, Logan Gunthorpe wrote:
>> Hi,
>>
>> Here's an updated version of the style fixes patchset. The differences
>> from v1 are:
>>
>> 1) Rebased onto Jon Mason's current NTB branch
>>
>> 2) Added two more patches to convert the print lines to use the ntb
>> device and not the pci one. This seems more sane, shortens a bunch
>> of lines and removes the need for temporaries.
>>
>> Note with (2): I've tried my best to ensure that statements that print
>> before the ntb device is registered still use the PCI device. However,
>> someone should probably review and test this as I don't have access
>> to all types of hardware to do that.
>>
>> Thanks,
>>
>> Logan
>>
>>
>>
>> Logan Gunthorpe (4):
>>   ntb_hw_amd: Style fixes: open code macros that just obfuscate code
>>   ntb_hw_intel: Style fixes: open code macros that just obfuscate code
>>   ntb_hw_amd: Print kernel messages with the ntb device not the pci one
>>   ntb_hw_intel: Print kernel messages with the ntb device not the pci
>>     one
>>
>>  drivers/ntb/hw/amd/ntb_hw_amd.c     |  60 +++++------
>>  drivers/ntb/hw/amd/ntb_hw_amd.h     |   3 -
>>  drivers/ntb/hw/intel/ntb_hw_intel.c | 192 +++++++++++++++++-------------------
>>  drivers/ntb/hw/intel/ntb_hw_intel.h |   3 -
>>  4 files changed, 124 insertions(+), 134 deletions(-)
>>
>> --
>> 2.1.4
>>
>
> --
> You received this message because you are subscribed to the Google Groups "linux-ntb" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-ntb+unsubscribe@xxxxxxxxxxxxxxxxx
> To post to this group, send email to linux-ntb@xxxxxxxxxxxxxxxxx
> To view this discussion on the web visit https://groups.google.com/d/msgid/linux-ntb/5ed3be6c-c94e-1ade-e4cd-a8e8293296cd%40deltatee.com.
> For more options, visit https://groups.google.com/d/optout.