| Message ID | 20210101132432.2785663-2-bert@biot.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v3,1/2] dt-bindings: spi: Realtek RTL838x/RTL839x SPI controller | expand |
On Fri, Jan 01, 2021 at 02:24:31PM +0100, Bert Vermeulen wrote: > + compatible: > + const: realtek,rtl-spi This is still just using rtl-spi as the compatible string, please address the feedback on the previous version. Please submit patches using subject lines reflecting the style for the subsystem, this makes it easier for people to identify relevant patches. Look at what existing commits in the area you're changing are doing and make sure your subject lines visually resemble what they're doing. There's no need to resubmit to fix this alone.
On 1/4/21 10:31 PM, Mark Brown wrote: > On Fri, Jan 01, 2021 at 02:24:31PM +0100, Bert Vermeulen wrote: > >> + compatible: >> + const: realtek,rtl-spi > > This is still just using rtl-spi as the compatible string, please > address the feedback on the previous version. Mark, The rtl prefix really is as close as it gets. This is being developed on RTL838x and RTL839x, but the driver very likely also works on RTL8196C and RTL93xx series. There's no difference to the SPI block in those SoCs. Do you really want a different per-series compatible when the "IP block" was just copy-pasted between series in hardware?
On Tue, Jan 05, 2021 at 09:43:59AM +0100, Bert Vermeulen wrote: > On 1/4/21 10:31 PM, Mark Brown wrote: > > This is still just using rtl-spi as the compatible string, please > > address the feedback on the previous version. > The rtl prefix really is as close as it gets. This is being developed on > RTL838x and RTL839x, but the driver very likely also works on RTL8196C and > RTL93xx series. There's no difference to the SPI block in those SoCs. > Do you really want a different per-series compatible when the "IP block" was > just copy-pasted between series in hardware? Yes.
diff --git a/Documentation/devicetree/bindings/spi/realtek,rtl-spi.yaml b/Documentation/devicetree/bindings/spi/realtek,rtl-spi.yaml new file mode 100644 index 000000000000..0d3f8448c9ab --- /dev/null +++ b/Documentation/devicetree/bindings/spi/realtek,rtl-spi.yaml @@ -0,0 +1,36 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/spi/realtek,rtl-spi.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Realtek RTL838x/RTL839x SPI controller + +maintainers: + - Bert Vermeulen <bert@biot.com> + - Birger Koblitz <mail@birger-koblitz.de> + +allOf: + - $ref: "spi-controller.yaml#" + +properties: + compatible: + const: realtek,rtl-spi + + reg: + maxItems: 1 + +required: + - compatible + - reg + +unevaluatedProperties: false + +examples: + - | + spi: spi@1200 { + compatible = "realtek,rtl-spi"; + reg = <0x1200 0x100>; + #address-cells = <1>; + #size-cells = <0>; + };
Signed-off-by: Bert Vermeulen <bert@biot.com> --- .../bindings/spi/realtek,rtl-spi.yaml | 36 +++++++++++++++++++ 1 file changed, 36 insertions(+) create mode 100644 Documentation/devicetree/bindings/spi/realtek,rtl-spi.yaml