| Message ID | 20240422065114.3185505-1-christoph.fritz@hexdev.de |
|---|---|
| Headers | show |
| Series | LIN Bus support for Linux | expand |
On 22/04/2024 08:51, Christoph Fritz wrote: > Add documentation of device tree bindings for serdev UART LIN-Bus > devices equipped with LIN transceivers. A nit, subject: drop second/last, redundant "dt bindings". The "dt-bindings" prefix is already stating that these are bindings. See also: https://elixir.bootlin.com/linux/v6.7-rc8/source/Documentation/devicetree/bindings/submitting-patches.rst#L18 > > Signed-off-by: Christoph Fritz <christoph.fritz@hexdev.de> > --- > .../bindings/net/can/linux,lin-serdev.yaml | 29 +++++++++++++++++++ > 1 file changed, 29 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml > > diff --git a/Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml b/Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml > new file mode 100644 > index 0000000000000..cb4e932ff249c > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml > @@ -0,0 +1,29 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/can/linux,lin-serdev.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Linux serdev LIN-Bus Support This looks like Linux binding, but we expect here description of hardware. > + > +description: | > + LIN-Bus support for UART devices equipped with LIN transceivers, > + utilizing the Serial Device Bus (serdev) interface. serdev is Linux thingy, AFAIR. Please describe the hardware. > + > + For more details on an adapter, visit: https://hexdev.de/hexlin#tty > + > +properties: > + compatible: > + const: linux,lin-serdev Feels confusing. Your link describes real hardware, but you wrote bindings for software construct. If you add this to DT, then it is hard-wired on the board, right? If so, how this could be a software construct? > + > +required: > + - compatible > + > +examples: > + - | > + &uart2 { & does not make much sense here. I think you wanted it to be serial bus, so serial. > + status = "okay"; Drop, it was not disabled anywhere. > + linbus { > + compatible = "linux,lin-serdev"; > + }; > + }; Best regards, Krzysztof
On Mon, 22 Apr 2024 08:51:08 +0200, Christoph Fritz wrote: > Add documentation of device tree bindings for serdev UART LIN-Bus > devices equipped with LIN transceivers. > > Signed-off-by: Christoph Fritz <christoph.fritz@hexdev.de> > --- > .../bindings/net/can/linux,lin-serdev.yaml | 29 +++++++++++++++++++ > 1 file changed, 29 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml: 'maintainers' is a required property hint: Metaschema for devicetree binding documentation from schema $id: http://devicetree.org/meta-schemas/base.yaml# /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/can/linux,lin-serdev.yaml: 'oneOf' conditional failed, one must be fixed: 'unevaluatedProperties' is a required property 'additionalProperties' is a required property hint: Either unevaluatedProperties or additionalProperties must be present from schema $id: http://devicetree.org/meta-schemas/core.yaml# Error: Documentation/devicetree/bindings/net/can/linux,lin-serdev.example.dts:18.9-15 syntax error FATAL ERROR: Unable to parse input tree make[2]: *** [scripts/Makefile.lib:427: Documentation/devicetree/bindings/net/can/linux,lin-serdev.example.dtb] Error 1 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1430: dt_binding_check] Error 2 make: *** [Makefile:240: __sub-make] Error 2 doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240422065114.3185505-6-christoph.fritz@hexdev.de The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 22/04/2024 08:51, Christoph Fritz wrote: > Add documentation of device tree bindings for serdev UART LIN-Bus > devices equipped with LIN transceivers. > > Signed-off-by: Christoph Fritz <christoph.fritz@hexdev.de> > --- Please test patches before sending... Best regards, Krzysztof
On 23. 04. 24, 11:33, Christoph Fritz wrote: >>> --- /dev/null >>> +++ b/include/net/lin.h >>> @@ -0,0 +1,97 @@ ... >>> +/* special ID descriptions for LIN */ >>> +#define LIN_ENHANCED_CKSUM_FLAG 0x00000100U >>> + >>> +static const unsigned char lin_id_parity_tbl[] = { >> >> This ends up in every translation unit you include lin.h into. Bad. > > This is also being used by a serial lin driver. But I guess most of the drivers have no need for this. Mhm, ... any ideas? If needs be, put it to a .c and keep an extern here. thanks,