lkml.org 
[lkml]   [2022]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3 2/2] dt-bindings: arm: qcom: Document additional sku6 for sc7180 pazquel
Hi,

On Thu, Jul 21, 2022 at 9:33 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 21/07/2022 15:37, Doug Anderson wrote:
> >
> > Not worth sending a new version for, but normally I expect the
> > bindings to be patch #1 and the dts change to be patch #2. In any
> > case:
> >
> > Reviewed-by: Douglas Anderson <dianders@chromium.org>
>
> I would say worth v4, because otherwise patches is not bisectable.

You're saying because `dtbs_check` will fail between the two? How does
flipping the order help? If `dtbs_check` needs to be bisectable then
these two need to be one patch, but I was always under the impression
that we wanted bindings patches separate from dts patches.

-Doug

\
 
 \ /
  Last update: 2022-07-21 18:45    [W:0.070 / U:0.772 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site