Re: [PATCH 05/10] dt-bindings: pwm: convert pwm-bcm2835 bindings to YAML

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

 



On Tue, Jun 13, 2023 at 11:02:40AM +0200, Stefan Wahren wrote:
> Hi,
> 
> Am 09.06.23 um 23:59 schrieb Rob Herring:
> > On Wed, Jun 07, 2023 at 09:56:02AM +0200, Uwe Kleine-König wrote:
> > > Hello,
> > > 
> > > On Sun, Jun 04, 2023 at 02:12:18PM +0200, Stefan Wahren wrote:
> > > > Convert the DT binding document for pwm-bcm2835 from .txt to YAML.
> > > 
> > > Both dt_binding_check and dtbs_check (for ARCH=arm) are happy, apart
> > > from errors like:
> > > 
> > > arch/arm/boot/dts/bcm2711-rpi-4-b.dtb: pwm@7e20c000: #pwm-cells:0:0: 3 was expected
> > >          From schema: Documentation/devicetree/bindings/pwm/pwm-bcm2835.yaml
> > > 
> > > which is fixed in patch #6. So:
> > > 
> > > Reviewed-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> > > 
> > > Who will pick up this patch? Is it supposed to go in via pwm or
> > > arm/broadcom? (I suggest the latter.)
> > 
> > The default is the subsystem (pwm). Why not here?
> 
> i think Uwe's concern is about the fact, that the series is late in cycle.
> So the chance that the fixing DTS patch 6 miss Linux 6.5 is very high.

My concern is only that nobody might feel responsible and think the
other party will pick up. Thierry didn't pick up patches yet for the
next merge window (or I missed it), so I guess it's not yet to late for
going in via pwm.

I suggest to let it go in via the arm/broadcom tree, to have the whole
series together.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature


[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