On Mon, Mar 19, 2018 at 1:19 AM, <skolluku@xxxxxxxxxxxxxx> wrote: > On 2018-03-18 18:19, Rob Herring wrote: >> >> On Wed, Mar 14, 2018 at 11:21:37AM +0530, Sravanthi Kollukuduru wrote: >>> >>> Remove DT entries of hw block offsets and other target specific catalog >>> information for SDM845. >> >> >> That is clear from the diff. The commit msg should answer why? >> > Will update the commit msg as below in the follow up patch : > "Currently, the downstream driver depends on the DT file for the hw offsets > and other target specific catalog information. > To align the driver with the upstream DT design, this information is now > removed from DT file and added in the driver source." Based on the answer below, write whatever you want because until it is intended for upstream it doesn't matter (to me at least). Though, writing good commit messages should perhaps be practiced. >>> >>> Signed-off-by: Sravanthi Kollukuduru <skolluku@xxxxxxxxxxxxxx> >>> --- >>> .../devicetree/bindings/display/msm/dpu.txt | 530 >>> --------------------- >> >> >> This file is not upstream. What tree is this for? >> > This is part of the qualcomm efforts to upstream the display controller > code. > The changes are hosted at > https://cgit.freedesktop.org/~seanpaul/dpu-staging/ Please make that abundantly clear in every patch for this so it is obvious to ignore it. Or just keep it on the msm/freedreno lists until you are submitting things upstream. If you have specific binding questions/issues before them, I'm happy to discuss, but otherwise folks have more than enough to review just for upstream. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
