Re: [PATCH v6] dt-bindings: imx-pata: Convert to dtschema

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

 



On Mon, Mar 11, 2024 at 9:39 AM Damien Le Moal <dlemoal@xxxxxxxxxx> wrote:
> It is simple: the commit message should always explain *WHAT* you did and
> *WHY*. This is to give some context to reviewers and to help with checking that
> your code actually does what you explained. This also helps with potential
> future issues with a change as the commit message remains in the git log history.
>
> Regardless of the version of your patch, always have the what & why explained
> in your commit message. This implies that the commit message must change if the
> patch content changes between versions. Keep in mind that the changelog added
> to a patch is lost when the patch is applied, but the commit message remains.

Thank you for your feedback and guidance.
Your advice regarding the necessity of explaining both the *WHAT* and
*WHY* behind each change is duly noted. Moving forward, I will ensure
that my commit messages provide comprehensive context to facilitate
smoother reviewing processes and to maintain a clear log history for
potential future issues.

Thanks & regards,
Animesh





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux