Re: [PATCH v5 0/2] Samsung Galaxy Z Fold5 initial support

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

 



Hi Krzysztof,

Thank you for your time and please excuse my lack of knowledge, reviewing the first reply I clearly see now the details, but i missed them initially, I suppose that due to the lack of knowledge in regards to the procedures here.

I am going to  generate a v6, thanks again for your valuable time spent helping me.


Best regards,

Alexandru Marc Serdeliuc


On 11/4/24 10:47, Krzysztof Kozlowski wrote:
On 11/04/2024 09:34, Krzysztof Kozlowski wrote:
On 11/04/2024 08:36, Alexandru Serdeliuc wrote:
Hi,

The list of changes  (changelog) from the cover is not what I should  add?

My patches received only two ACK tags, on V3 and on the initial request
(v1), I was not able to identify any other, I added them to their place
in the change log

...
- v3
    . added b4 version 3
    . removed address and size cells in device description
    Acked-by: Rob Herring<robh@xxxxxxxxxx>
...
- v1
    . The initial request was split in two patches sent due to the following checkpatch warning, was requested to re send them together:
      WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
    Acked-by: Krzysztof Kozlowski<krzysztof.kozlowski@xxxxxxxxxx>

I suppose that adding them to their place in change log is wrong, I
should create a v6 and put them at the end of the cover letter? Or how
to proceed?
Please don't top post. If you add them to the changelog, how are they
going to be effective? Please apply your patch (e.g. b4 shazam) and look
for them...

Submitting patches explains where to add tags. Look at other mailings.
And finally: why even bothering about this if b4 does it for you?

BTW, in reply to your first posting I gave you detailed instruction how
to proceed with tags. Let me quote:

"Please add Acked-by/Reviewed-by/Tested-by tags when posting new
versions, under or above your Signed-off-by tag"

I think it is clear where you should add it. I gave this instruction on
purpose so we will avoid this mess...

Drop all invalid acks from cover letter and send v6 with proper tags
places in mentioned place.

Best regards,
Krzysztof





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux