From: Bhanu Prakash Maiya <[email protected]>
Add a compatible string to support the UART implementation of the cros
ec interface. The driver does not support the reg and interrupt
properties, so exempt them from being required for UART compatible nodes.
Signed-off-by: Bhanu Prakash Maiya <[email protected]>
Co-developed-by: Mark Hasemeyer <[email protected]>
Signed-off-by: Mark Hasemeyer <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
---
Changes in v10:
- No change
Changes in v9:
- Rebase onto for-next
- Update node-name from uart0 to serial
- Remove reg and interrupt property requirement for UART compatible
nodes.
- Update authorship
Changes in v8:
- Update commit message
Changes in v7:
- No change
Changes in v6:
- No change
Changes in v5:
- No change
Changes in v4:
- Changes in commit message.
Changes in v3:
- Rebased changes on google,cros-ec.yaml
Changes in v2:
- No change
---
.../bindings/mfd/google,cros-ec.yaml | 20 +++++++++++++++++++
1 file changed, 20 insertions(+)
diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
index 3d5efa5578d1b..cdf1d719efe9d 100644
--- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
+++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
@@ -33,6 +33,9 @@ properties:
- description:
For implementations of the EC connected through RPMSG.
const: google,cros-ec-rpmsg
+ - description:
+ For implementations of the EC connected through UART.
+ const: google,cros-ec-uart
controller-data: true
@@ -187,6 +190,15 @@ allOf:
properties:
mediatek,rpmsg-name: false
+ - if:
+ properties:
+ compatible:
+ not:
+ contains:
+ enum:
+ - google,cros-ec-rpmsg
+ - google,cros-ec-uart
+ then:
required:
- reg
- interrupts
@@ -299,4 +311,12 @@ examples:
vdd-supply = <&pp3300_fp_mcu>;
};
};
+
+ # Example for UART
+ - |
+ serial {
+ cros-ec {
+ compatible = "google,cros-ec-uart";
+ };
+ };
...
--
2.39.0.rc1.256.g54fd8350bd-goog
On 07/12/2022 18:40, Mark Hasemeyer wrote:
> From: Bhanu Prakash Maiya <[email protected]>
>
> Add a compatible string to support the UART implementation of the cros
> ec interface. The driver does not support the reg and interrupt
> properties, so exempt them from being required for UART compatible nodes.
>
> Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> Co-developed-by: Mark Hasemeyer <[email protected]>
> Signed-off-by: Mark Hasemeyer <[email protected]>
> Acked-by: Krzysztof Kozlowski <[email protected]>
Use subject prefixes matching the subsystem (git log --oneline -- ...)
and device. Currently it looks like you change ENTIRE MFD subsystem.
> ---
>
> Changes in v10:
> - No change
>
Best regards,
Krzysztof
On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> From: Bhanu Prakash Maiya <[email protected]>
>
> Add a compatible string to support the UART implementation of the cros
> ec interface. The driver does not support the reg and interrupt
> properties, so exempt them from being required for UART compatible nodes.
>
> Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> Co-developed-by: Mark Hasemeyer <[email protected]>
> Signed-off-by: Mark Hasemeyer <[email protected]>
> Acked-by: Krzysztof Kozlowski <[email protected]>
> ---
>
> Changes in v10:
> - No change
>
> Changes in v9:
> - Rebase onto for-next
> - Update node-name from uart0 to serial
> - Remove reg and interrupt property requirement for UART compatible
> nodes.
> - Update authorship
>
> Changes in v8:
> - Update commit message
>
> Changes in v7:
> - No change
>
> Changes in v6:
> - No change
>
> Changes in v5:
> - No change
>
> Changes in v4:
> - Changes in commit message.
>
> Changes in v3:
> - Rebased changes on google,cros-ec.yaml
>
> Changes in v2:
> - No change
>
> ---
> .../bindings/mfd/google,cros-ec.yaml | 20 +++++++++++++++++++
> 1 file changed, 20 insertions(+)
I changed the subject line and applied the patch, thanks.
--
Lee Jones [李琼斯]
On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
>
> > From: Bhanu Prakash Maiya <[email protected]>
> >
> > Add a compatible string to support the UART implementation of the cros
> > ec interface. The driver does not support the reg and interrupt
> > properties, so exempt them from being required for UART compatible nodes.
> >
> > Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> > Co-developed-by: Mark Hasemeyer <[email protected]>
> > Signed-off-by: Mark Hasemeyer <[email protected]>
> > Acked-by: Krzysztof Kozlowski <[email protected]>
> > ---
> >
> > Changes in v10:
> > - No change
> >
[...]
>
> I changed the subject line and applied the patch, thanks.
Pardon me. I didn't know you would pick the patch so that I have queued it
into chrome-platform[1]. Would you like me to drop the commit?
[1]: https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git/commit/?h=for-next&id=5be8cd61dcddfaee5d92072073672037154e76f5
On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
> On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> > On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> >
> > > From: Bhanu Prakash Maiya <[email protected]>
> > >
> > > Add a compatible string to support the UART implementation of the cros
> > > ec interface. The driver does not support the reg and interrupt
> > > properties, so exempt them from being required for UART compatible nodes.
> > >
> > > Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> > > Co-developed-by: Mark Hasemeyer <[email protected]>
> > > Signed-off-by: Mark Hasemeyer <[email protected]>
> > > Acked-by: Krzysztof Kozlowski <[email protected]>
> > > ---
> > >
> > > Changes in v10:
> > > - No change
> > >
> [...]
> >
> > I changed the subject line and applied the patch, thanks.
>
> Pardon me. I didn't know you would pick the patch so that I have queued it
> into chrome-platform[1]. Would you like me to drop the commit?
>
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git/commit/?h=for-next&id=5be8cd61dcddfaee5d92072073672037154e76f5
--
Lee Jones [李琼斯]
On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
> On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> > On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> >
> > > From: Bhanu Prakash Maiya <[email protected]>
> > >
> > > Add a compatible string to support the UART implementation of the cros
> > > ec interface. The driver does not support the reg and interrupt
> > > properties, so exempt them from being required for UART compatible nodes.
> > >
> > > Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> > > Co-developed-by: Mark Hasemeyer <[email protected]>
> > > Signed-off-by: Mark Hasemeyer <[email protected]>
> > > Acked-by: Krzysztof Kozlowski <[email protected]>
> > > ---
> > >
> > > Changes in v10:
> > > - No change
> > >
> [...]
> >
> > I changed the subject line and applied the patch, thanks.
>
> Pardon me. I didn't know you would pick the patch so that I have queued it
> into chrome-platform[1]. Would you like me to drop the commit?
Yes, please do.
You should gain permission from the maintainer before you apply patches.
% scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD))
Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
Krzysztof Kozlowski <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
Matthias Brugger <[email protected]> (maintainer:ARM/Mediatek SoC support)
Why do I get the feeling we've had this conversation before?
--
Lee Jones [李琼斯]
On Thu, Jan 05, 2023 at 02:24:26PM +0000, Lee Jones wrote:
> On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
>
> > On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> > > On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> > >
> > > > From: Bhanu Prakash Maiya <[email protected]>
> > > >
> > > > Add a compatible string to support the UART implementation of the cros
> > > > ec interface. The driver does not support the reg and interrupt
> > > > properties, so exempt them from being required for UART compatible nodes.
> > > >
> > > > Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> > > > Co-developed-by: Mark Hasemeyer <[email protected]>
> > > > Signed-off-by: Mark Hasemeyer <[email protected]>
> > > > Acked-by: Krzysztof Kozlowski <[email protected]>
> > > > ---
> > > >
> > > > Changes in v10:
> > > > - No change
> > > >
> > [...]
> > >
> > > I changed the subject line and applied the patch, thanks.
> >
> > Pardon me. I didn't know you would pick the patch so that I have queued it
> > into chrome-platform[1]. Would you like me to drop the commit?
>
> Yes, please do.
Dropped in chrome-platform.
> You should gain permission from the maintainer before you apply patches.
>
> % scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
> Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD))
> Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> Krzysztof Kozlowski <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> Matthias Brugger <[email protected]> (maintainer:ARM/Mediatek SoC support)
I see. I guess for the case, I should get both MFD and DT bindings
maintainers' Acks before applying the patch.
On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
> On Thu, Jan 05, 2023 at 02:24:26PM +0000, Lee Jones wrote:
> > On Thu, 05 Jan 2023, Tzung-Bi Shih wrote:
> >
> > > On Wed, Jan 04, 2023 at 04:08:24PM +0000, Lee Jones wrote:
> > > > On Wed, 07 Dec 2022, Mark Hasemeyer wrote:
> > > >
> > > > > From: Bhanu Prakash Maiya <[email protected]>
> > > > >
> > > > > Add a compatible string to support the UART implementation of the cros
> > > > > ec interface. The driver does not support the reg and interrupt
> > > > > properties, so exempt them from being required for UART compatible nodes.
> > > > >
> > > > > Signed-off-by: Bhanu Prakash Maiya <[email protected]>
> > > > > Co-developed-by: Mark Hasemeyer <[email protected]>
> > > > > Signed-off-by: Mark Hasemeyer <[email protected]>
> > > > > Acked-by: Krzysztof Kozlowski <[email protected]>
> > > > > ---
> > > > >
> > > > > Changes in v10:
> > > > > - No change
> > > > >
> > > [...]
> > > >
> > > > I changed the subject line and applied the patch, thanks.
> > >
> > > Pardon me. I didn't know you would pick the patch so that I have queued it
> > > into chrome-platform[1]. Would you like me to drop the commit?
> >
> > Yes, please do.
>
> Dropped in chrome-platform.
>
> > You should gain permission from the maintainer before you apply patches.
> >
> > % scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
> > Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD))
> > Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > Krzysztof Kozlowski <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > Matthias Brugger <[email protected]> (maintainer:ARM/Mediatek SoC support)
>
> I see. I guess for the case, I should get both MFD and DT bindings
> maintainers' Acks before applying the patch.
Yes. The situation isn't overly clear and differs from subsystem to
subsystem and maintainer to maintainer. In the MFD case, Rob and
Krzysztof conduct the first reviews. 99 times out of 100, if they're
happy, I'm happy and I merge the changes via the MFD tree. Same goes
for Backlight and more recently, LEDs.
--
Lee Jones [李琼斯]
On Thu, Jan 05, 2023 at 04:27:32PM +0000, Lee Jones wrote:
> > > You should gain permission from the maintainer before you apply patches.
> > >
> > > % scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
> > > Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD))
> > > Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > > Krzysztof Kozlowski <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > > Matthias Brugger <[email protected]> (maintainer:ARM/Mediatek SoC support)
> >
> > I see. I guess for the case, I should get both MFD and DT bindings
> > maintainers' Acks before applying the patch.
>
> Yes. The situation isn't overly clear and differs from subsystem to
> subsystem and maintainer to maintainer. In the MFD case, Rob and
> Krzysztof conduct the first reviews. 99 times out of 100, if they're
> happy, I'm happy and I merge the changes via the MFD tree. Same goes
> for Backlight and more recently, LEDs.
I see. Apologize for that and I will pay more attention next time when
handling cross-subsystem patches.
On Wed, Jan 4, 2023 at 9:08 AM Lee Jones <[email protected]> wrote:
> I changed the subject line and applied the patch, thanks.
Hey Lee, a couple comments/questions:
1. The subject line of this patch was updated in v11 of the series.
2. What tree was the patch applied to? I don't see the changes in linux-next.
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
On Thu, Jan 5, 2023 at 11:08 AM Tzung-Bi Shih <[email protected]> wrote:
>
> On Thu, Jan 05, 2023 at 04:27:32PM +0000, Lee Jones wrote:
> > > > You should gain permission from the maintainer before you apply patches.
> > > >
> > > > % scripts/get_maintainer.pl -f Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
> > > > Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD))
> > > > Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > > > Krzysztof Kozlowski <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> > > > Matthias Brugger <[email protected]> (maintainer:ARM/Mediatek SoC support)
> > >
> > > I see. I guess for the case, I should get both MFD and DT bindings
> > > maintainers' Acks before applying the patch.
> >
> > Yes. The situation isn't overly clear and differs from subsystem to
> > subsystem and maintainer to maintainer. In the MFD case, Rob and
> > Krzysztof conduct the first reviews. 99 times out of 100, if they're
> > happy, I'm happy and I merge the changes via the MFD tree. Same goes
> > for Backlight and more recently, LEDs.
>
> I see. Apologize for that and I will pay more attention next time when
> handling cross-subsystem patches.
On Mon, 09 Jan 2023, Mark Hasemeyer wrote:
> On Wed, Jan 4, 2023 at 9:08 AM Lee Jones <[email protected]> wrote:
> > I changed the subject line and applied the patch, thanks.
>
> Hey Lee, a couple comments/questions:
> 1. The subject line of this patch was updated in v11 of the series.
> 2. What tree was the patch applied to? I don't see the changes in linux-next.
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
Thanks for flagging. You will do tomorrow.
--
Lee Jones [李琼斯]