On Tue, Feb 15, 2022 at 09:02:25AM +0100, Krzysztof Kozlowski wrote: > On 15/02/2022 08:59, Krzysztof Kozlowski wrote: > > On Tue, 15 Feb 2022 at 08:40, Uwe Kleine-König > > wrote: > >> > >> Hello, > >> > >> [dropped Anson Huang and Yash Shah from Cc: which were not reachable for > >> my last mail] > >> > >> On Mon, Feb 14, 2022 at 10:21:39PM +0100, Krzysztof Kozlowski wrote: > >>> Hi, > >>> > >>> Changes since v1: > >>> 1. Add tags. > >>> 2. Adjust subject (Uwe). > >> > >> However you only took a part of my suggestion ... > >> > >>> Krzysztof Kozlowski (15): > >>> dt-bindings: pwm: allwinner,sun4i-a10: Include generic pwm schema > >>> dt-bindings: pwm: imx: Include generic pwm schema > >>> dt-bindings: pwm: intel,lgm: Include generic pwm schema > >>> dt-bindings: pwm: iqs620a: Include generic pwm schema > >>> dt-bindings: pwm: mxs: Include generic pwm schema > >>> dt-bindings: pwm: rockchip: Include generic pwm schema > >>> dt-bindings: pwm: sifive: Include generic pwm schema > >>> dt-bindings: pwm: renesas,pwm: Include generic pwm schema > >>> dt-bindings: pwm: toshiba,visconti: Include generic pwm schema > >>> dt-bindings: pwm: brcm,bcm7038: Do not require pwm-cells twice > >>> dt-bindings: pwm: intel,keembay: Do not require pwm-cells twice > >> > >> ... The actual patch has a space after the comma, I like this variant > >> without comma better as this is a compatible string. > > > > I am confused. My patch does not have comma after space. Your reply > > had such in the subject, but not in the proposed new subject you wrote > > in msg, so I left it as is. Without comma. If you still see comma, it > > is something with your mail client. > > > > See: > > https://lore.kernel.org/linux-devicetree/20220214212154.8853-12-krzysztof.kozlowski@canonical.com/T/#u > > > > Also reply from Vijayakannan does not have comma: > > https://lore.kernel.org/linux-devicetree/20220214081605.161394-11-krzysztof.kozlowski@canonical.com/T/#m80af695f2c751341bc971114aefa00ccc929a3ec Strange: I have this mail four times in my mailboxes (via linux-arm-kernel, linux-pwm, kernel@pengutronix.de and directly). In the two latter the Subject line is broken in two: Subject: [PATCH v2 08/15] dt-bindings: pwm: renesas, pwm: Include generic pwm schema So this is indeed a problem near my side. > Just one more clarification - you proposed also to remove "pwm:" > middle-prefix. Shall I remove it from every patch and use instead full > file name? I did that to save some vertical space in the Subject line. I don't feel strong here. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |