Re: Please pick up i2c-xlp9xx driver patches

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dropping openipmi list cross-post...

On Thu, Feb 1, 2018 at 12:12 AM, Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote:
> Plaese note that I took over I2C from Jean in late 2012...

Oh.  Jean is still listed in the MAINTAINERS file and I remember
working with him for years on i2c and lm_sensors related codes and
patches.

>> I tested the patches and they fix the ipmi_ssif detection issue I had.
>> So I'm willing to add "Tested-by" to the patches if that helps.
>
> Ehrm, why didn't you do this right away?

Because I only just learned about it 2 days ago.  It took Cavium a
while to get me a pointer to the patch they said would fix my problem.
Said problem I've only had for a couple of weeks since it's brand new
(to me) hardware.

> This helps _a lot_! With these
> tags, I might have considered them earlier. But without tags, they
> surely needed a full development cycle to avoid regressions. Which means
> merge window for v4.17. With tags, I could now consider them for v4.16.

That's great to hear.  I haven't had a chance to contribute much since
the change to Git, so I'm not familiar with the "local customs" and
procedures to add those tags.  Can you help by being specific about
what you need from me?

While the patches did apply to the latest 4.15 kernel tree, there was
some fuzz and I think it would be good to rebase or go over them to
make sure that all the code bits are put in the right places.  I guess
that falls to me to do, but I'm worried we'll miss the 4.16 window
which I would really like to see happen.

> Rule of thumb: If you need patches, show your interest.

Understood and I am...

Phil P.


-- 
Philip Pokorny, RHCE
Chief Technology Officer
PENGUIN COMPUTING, Inc
www.penguincomputing.com

Changing the world through technical innovation



[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux