lkml.org 
[lkml]   [2023]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v1 0/6] arm64: qcom: sa8775p: add support for PCIe
    From
    On 6.07.2023 14:09, Krzysztof Kozlowski wrote:
    > On 06/07/2023 13:12, Manivannan Sadhasivam wrote:
    >>>> Please note that the dts patches should come before driver patches.
    >>>>
    >>>
    >>> Why? DTS is always independent thus usually put at the end of patchset
    >>> or better separate. It is the first time I hear that DTS should be
    >>> before driver.
    >>>
    >>
    >> This is what I was suggested by Rob during my initial days and I've been
    >> following this pattern since then. If that's not the case, I have no issues.
    >
    > I really doubt Rob ever suggested that. He probably suggested that
    > bindings must come before user, but not the DTS.
    Besides, the flow in this patchset is very logical from the Linux POV

    1. describe the hardware in bindings
    2. implement the drivers, conforming to the description
    3. describe the hardware in the device tree to ensure the (now
    existing) driver probes

    Konrad
    >
    > Best regards,
    > Krzysztof
    >

    \
     
     \ /
      Last update: 2023-07-06 14:14    [W:3.535 / U:2.488 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site