On Wed, Apr 08, 2020 at 12:51:43AM +0300, Andi Shyti wrote:

Hi Stephan,

On Sun, Apr 05, 2020 at 07:09:04PM +0200, Stephan Gerhold wrote:

quoted

The mms114 driver now supports MMS345L; document the
melfas,mms345l binding that is used for it.

Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Andi Shyti <redacted>
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>

just one nitpick, the signing should be sorted in chronological
order, I see that you reverted it.

You first signed it as the author, then I reviewed it and the Rob
acked it, so that it should be:

  Signed-off-by: Stephan Gerhold [off-list ref]
  Reviewed-by: Andi Shyti [off-list ref]
  Acked-by: Rob Herring [off-list ref]

you reversed it.

I didn't really "reverse" it, I didn't know the order of these tags is
important at all. :) Is this documented somewhere?

Since I have added the tags and re-sent the patch,
I thought it would make sense have my Signed-off-by last.

I suppose this can be easily fixed up if Dmitry wants to apply the
patch. But if I should re-send with the order changed in some way,
just let me know. :)

Other than that, I'm sorry I couldn't help you but I haven't
received answers from Samsung about getting the prototypes.
However I don't see reason for not applying the patch.

It's fine, don't worry :)

I just think at this point we should either take my patch
(and preserve existing behavior for all currently supported devices),
or accept the potential breakage when refactoring the driver
(because seemingly no-one is able to test it on the other versions
at the moment).

Thanks,
Stephan