| Message ID | 20200828130602.42203-1-andre.przywara@arm.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | dt-bindings: Convert SP805 to Json-schema (and fix users) | expand |
On 8/28/20 6:05 AM, Andre Przywara wrote: > This is an attempt to convert the SP805 watchdog DT binding to yaml. > This is done in the first patch, the remaining nine fix some DT users. > > I couldn't test any of those DT files on actual machines, but tried > to make the changes in a way that would be transparent to at least the > Linux driver. The only other SP805 DT user I could find is U-Boot, which > seems to only use a very minimal subset of the binding (just the first > clock). > I only tried to fix those DTs that were easily and reliably fixable. > AFAICT, a missing primecell compatible string, for instance, would > prevent the Linux driver from probing the device at all, so I didn't > dare to touch those DTs at all. Missing clocks are equally fatal. What is the plan for merging this series? Should Rob pick up all changes or since those are non critical changes, should we just leave it to the SoC maintainers to pick up the changes in their tree? Likewise for the SP804 timer series, what's the plan?
On Fri, Aug 28, 2020 at 1:34 PM Florian Fainelli <f.fainelli@gmail.com> wrote: > > On 8/28/20 6:05 AM, Andre Przywara wrote: > > This is an attempt to convert the SP805 watchdog DT binding to yaml. > > This is done in the first patch, the remaining nine fix some DT users. > > > > I couldn't test any of those DT files on actual machines, but tried > > to make the changes in a way that would be transparent to at least the > > Linux driver. The only other SP805 DT user I could find is U-Boot, which > > seems to only use a very minimal subset of the binding (just the first > > clock). > > I only tried to fix those DTs that were easily and reliably fixable. > > AFAICT, a missing primecell compatible string, for instance, would > > prevent the Linux driver from probing the device at all, so I didn't > > dare to touch those DTs at all. Missing clocks are equally fatal. > > What is the plan for merging this series? Should Rob pick up all changes > or since those are non critical changes, should we just leave it to the > SoC maintainers to pick up the changes in their tree? I don't take .dts files. Either subarch maintainers can pick up individual patches or send a PR to SoC maintainers. Rob
On 8/28/20 2:28 PM, Rob Herring wrote: > On Fri, Aug 28, 2020 at 1:34 PM Florian Fainelli <f.fainelli@gmail.com> wrote: >> >> On 8/28/20 6:05 AM, Andre Przywara wrote: >>> This is an attempt to convert the SP805 watchdog DT binding to yaml. >>> This is done in the first patch, the remaining nine fix some DT users. >>> >>> I couldn't test any of those DT files on actual machines, but tried >>> to make the changes in a way that would be transparent to at least the >>> Linux driver. The only other SP805 DT user I could find is U-Boot, which >>> seems to only use a very minimal subset of the binding (just the first >>> clock). >>> I only tried to fix those DTs that were easily and reliably fixable. >>> AFAICT, a missing primecell compatible string, for instance, would >>> prevent the Linux driver from probing the device at all, so I didn't >>> dare to touch those DTs at all. Missing clocks are equally fatal. >> >> What is the plan for merging this series? Should Rob pick up all changes >> or since those are non critical changes, should we just leave it to the >> SoC maintainers to pick up the changes in their tree? > > I don't take .dts files. Either subarch maintainers can pick up > individual patches or send a PR to SoC maintainers. OK, so we are fine, to say make sure this all lands in v5.10-rc1 at some point and the warnings should no longer exist by then?
On 28/08/2020 22:32, Florian Fainelli wrote: Hi, Florian, thanks for queueing the Broadcom specific patches! > On 8/28/20 2:28 PM, Rob Herring wrote: >> On Fri, Aug 28, 2020 at 1:34 PM Florian Fainelli <f.fainelli@gmail.com> wrote: >>> >>> On 8/28/20 6:05 AM, Andre Przywara wrote: >>>> This is an attempt to convert the SP805 watchdog DT binding to yaml. >>>> This is done in the first patch, the remaining nine fix some DT users. >>>> >>>> I couldn't test any of those DT files on actual machines, but tried >>>> to make the changes in a way that would be transparent to at least the >>>> Linux driver. The only other SP805 DT user I could find is U-Boot, which >>>> seems to only use a very minimal subset of the binding (just the first >>>> clock). >>>> I only tried to fix those DTs that were easily and reliably fixable. >>>> AFAICT, a missing primecell compatible string, for instance, would >>>> prevent the Linux driver from probing the device at all, so I didn't >>>> dare to touch those DTs at all. Missing clocks are equally fatal. >>> >>> What is the plan for merging this series? Should Rob pick up all changes >>> or since those are non critical changes, should we just leave it to the >>> SoC maintainers to pick up the changes in their tree? >> >> I don't take .dts files. Either subarch maintainers can pick up >> individual patches or send a PR to SoC maintainers. > > OK, so we are fine, to say make sure this all lands in v5.10-rc1 at some > point and the warnings should no longer exist by then? So yes, I would be very grateful if subsystem maintainers take this at their discretion. For once, I didn't actually change anything in the binding, so most things were already slightly wrong according to the .txt binding, just nobody realised or cared. So those .dts files changes are actually independent and justified even without patch 01/10. Secondly, there are already so many warnings in many .dts files at the moment, that (in the worst case) a few more - for a brief period of time - do not really matter. But at the end it will improve the situation. Rob, if you are fine with the actual binding, I would try to pursue the remaining subsystem maintainers to get the .dts changes merged. Thanks, Andre.
On Fri, Aug 28, 2020 at 9:34 PM Florian Fainelli <f.fainelli@gmail.com> wrote: > On 8/28/20 6:05 AM, Andre Przywara wrote: > What is the plan for merging this series? Should Rob pick up all changes > or since those are non critical changes, should we just leave it to the > SoC maintainers to pick up the changes in their tree? What about André just send a pull request to the ARM SoC maintainers for the whole thing? Yours, Linus Walleij
On 9/4/2020 1:58 AM, Linus Walleij wrote: > On Fri, Aug 28, 2020 at 9:34 PM Florian Fainelli <f.fainelli@gmail.com> wrote: >> On 8/28/20 6:05 AM, Andre Przywara wrote: > >> What is the plan for merging this series? Should Rob pick up all changes >> or since those are non critical changes, should we just leave it to the >> SoC maintainers to pick up the changes in their tree? > > What about André just send a pull request to the ARM SoC maintainers > for the whole thing? I already applied some of the patches, if we got that route please CC me so I can drop them from my local queue. Thanks
On 04/09/2020 16:29, Florian Fainelli wrote: Hi, > On 9/4/2020 1:58 AM, Linus Walleij wrote:>> On Fri, Aug 28, 2020 at 9:34 PM Florian Fainelli >> <f.fainelli@gmail.com> wrote: >>> On 8/28/20 6:05 AM, Andre Przywara wrote: >> >>> What is the plan for merging this series? Should Rob pick up all changes >>> or since those are non critical changes, should we just leave it to the >>> SoC maintainers to pick up the changes in their tree? >> >> What about André just send a pull request to the ARM SoC maintainers >> for the whole thing? > > I already applied some of the patches, if we got that route please CC me > so I can drop them from my local queue. Thanks I would for sure drop these from any PR. Rob, are you happy with the actual binding conversion? If you are willing to take it as it is (Viresh has already acked), I could then split off the DT fixes and either chase the maintainers or send ARM SoC a PR. But this really depends on the binding being good. Cheers, Andre.
On 9/4/2020 8:35 AM, André Przywara wrote: > On 04/09/2020 16:29, Florian Fainelli wrote: > > Hi, > >> On 9/4/2020 1:58 AM, Linus Walleij wrote:>> On Fri, Aug 28, 2020 at 9:34 PM Florian Fainelli >>> <f.fainelli@gmail.com> wrote: >>>> On 8/28/20 6:05 AM, Andre Przywara wrote: >>> >>>> What is the plan for merging this series? Should Rob pick up all changes >>>> or since those are non critical changes, should we just leave it to the >>>> SoC maintainers to pick up the changes in their tree? >>> >>> What about André just send a pull request to the ARM SoC maintainers >>> for the whole thing? >> >> I already applied some of the patches, if we got that route please CC me >> so I can drop them from my local queue. Thanks > > I would for sure drop these from any PR. > > Rob, are you happy with the actual binding conversion? If you are > willing to take it as it is (Viresh has already acked), I could then > split off the DT fixes and either chase the maintainers or send ARM SoC > a PR. But this really depends on the binding being good. We had discussed this in an another leg of this thread that starts here: https://lore.kernel.org/linux-devicetree/CAL_JsqKvcGAotS6xL7pu+wM8X33PLCQCuoaXYmWrA3j3OdoR5A@mail.gmail.com/ -- Florian
On Fri, 28 Aug 2020 14:05:52 +0100, Andre Przywara wrote: > This is an attempt to convert the SP805 watchdog DT binding to yaml. > This is done in the first patch, the remaining nine fix some DT users. > > I couldn't test any of those DT files on actual machines, but tried > to make the changes in a way that would be transparent to at least the > Linux driver. The only other SP805 DT user I could find is U-Boot, which > seems to only use a very minimal subset of the binding (just the first > clock). > I only tried to fix those DTs that were easily and reliably fixable. > AFAICT, a missing primecell compatible string, for instance, would > prevent the Linux driver from probing the device at all, so I didn't > dare to touch those DTs at all. Missing clocks are equally fatal. > > [...] I have picked 2 patches for Arm Ltd boards/models. Applied to sudeep.holla/linux (for-next/juno), thanks! [1/2] (korg_sudeep/for-next/juno, for-next/juno) arm64: dts: arm: Fix SP805 clock-names https://git.kernel.org/sudeep.holla/c/b83ded8a31 [2/2] ARM: dts: arm: Fix SP805 clocks https://git.kernel.org/sudeep.holla/c/a894c6dd56 -- Regards, Sudeep