From: Jonathan Cameron <jic23@kernel.org> To: Andrej Picej <andrej.picej@norik.com> Cc: haibo.chen@nxp.com, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, lars@metafoo.de, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, robh@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, upstream@lists.phytec.de Subject: Re: [PATCH 0/2] i.MX93 ADC calibration settings Date: Sun, 24 Mar 2024 13:55:59 +0000 [thread overview] Message-ID: <20240324135559.1640551d@jic23-huawei> (raw) In-Reply-To: <20240320100407.1639082-1-andrej.picej@norik.com> On Wed, 20 Mar 2024 11:04:04 +0100 Andrej Picej <andrej.picej@norik.com> wrote: > Hi all, > > we had some problems with failing ADC calibration on the i.MX93 boards. > Changing default calibration settings fixed this. The board where this > patches are useful is not yet upstream but will be soon (hopefully). Tell us more. My initial instinct is that this shouldn't be board specific. What's the trade off we are making here? Time vs precision of calibration or something else? If these are set to a level by default that doesn't work for our board, maybe we should just change them for all devices? Jonathan > > Since we had these patches laying around we thought they might also be > useful for someone else. > > Best regards, > Andrej > > Andrej Picej (2): > iio: adc: imx93: Make calibration properties configurable > dt-bindings: iio: adc: nxp,imx93-adc.yaml: Add calibration properties > > .../bindings/iio/adc/nxp,imx93-adc.yaml | 15 +++++ > drivers/iio/adc/imx93_adc.c | 66 +++++++++++++++++-- > 2 files changed, 76 insertions(+), 5 deletions(-) >
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <jic23@kernel.org> To: Andrej Picej <andrej.picej@norik.com> Cc: haibo.chen@nxp.com, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, lars@metafoo.de, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, robh@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, upstream@lists.phytec.de Subject: Re: [PATCH 0/2] i.MX93 ADC calibration settings Date: Sun, 24 Mar 2024 13:55:59 +0000 [thread overview] Message-ID: <20240324135559.1640551d@jic23-huawei> (raw) In-Reply-To: <20240320100407.1639082-1-andrej.picej@norik.com> On Wed, 20 Mar 2024 11:04:04 +0100 Andrej Picej <andrej.picej@norik.com> wrote: > Hi all, > > we had some problems with failing ADC calibration on the i.MX93 boards. > Changing default calibration settings fixed this. The board where this > patches are useful is not yet upstream but will be soon (hopefully). Tell us more. My initial instinct is that this shouldn't be board specific. What's the trade off we are making here? Time vs precision of calibration or something else? If these are set to a level by default that doesn't work for our board, maybe we should just change them for all devices? Jonathan > > Since we had these patches laying around we thought they might also be > useful for someone else. > > Best regards, > Andrej > > Andrej Picej (2): > iio: adc: imx93: Make calibration properties configurable > dt-bindings: iio: adc: nxp,imx93-adc.yaml: Add calibration properties > > .../bindings/iio/adc/nxp,imx93-adc.yaml | 15 +++++ > drivers/iio/adc/imx93_adc.c | 66 +++++++++++++++++-- > 2 files changed, 76 insertions(+), 5 deletions(-) > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-03-24 13:56 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-20 10:04 [PATCH 0/2] i.MX93 ADC calibration settings Andrej Picej 2024-03-20 10:04 ` Andrej Picej 2024-03-20 10:04 ` [PATCH 1/2] iio: adc: imx93: Make calibration properties configurable Andrej Picej 2024-03-20 10:04 ` Andrej Picej 2024-03-24 14:02 ` Jonathan Cameron 2024-03-24 14:02 ` Jonathan Cameron 2024-03-20 10:04 ` [PATCH 2/2] dt-bindings: iio: adc: nxp,imx93-adc.yaml: Add calibration properties Andrej Picej 2024-03-20 10:04 ` Andrej Picej 2024-03-20 10:26 ` Krzysztof Kozlowski 2024-03-20 10:26 ` Krzysztof Kozlowski 2024-03-20 12:05 ` Andrej Picej 2024-03-20 12:05 ` Andrej Picej 2024-03-20 12:15 ` Krzysztof Kozlowski 2024-03-20 12:15 ` Krzysztof Kozlowski 2024-03-22 7:39 ` Andrej Picej 2024-03-22 7:39 ` Andrej Picej 2024-03-22 8:14 ` Krzysztof Kozlowski 2024-03-22 8:14 ` Krzysztof Kozlowski 2024-03-22 9:58 ` Andrej Picej 2024-03-22 9:58 ` Andrej Picej 2024-03-24 13:54 ` Jonathan Cameron 2024-03-24 13:54 ` Jonathan Cameron 2024-03-25 9:58 ` Krzysztof Kozlowski 2024-03-25 9:58 ` Krzysztof Kozlowski 2024-03-25 14:38 ` Jonathan Cameron 2024-03-25 14:38 ` Jonathan Cameron 2024-03-20 21:41 ` Rob Herring 2024-03-20 21:41 ` Rob Herring 2024-03-22 6:47 ` kernel test robot 2024-03-22 6:47 ` kernel test robot 2024-03-24 13:55 ` Jonathan Cameron [this message] 2024-03-24 13:55 ` [PATCH 0/2] i.MX93 ADC calibration settings Jonathan Cameron 2024-03-25 8:32 ` Andrej Picej 2024-03-25 8:32 ` Andrej Picej 2024-03-25 8:55 ` [Upstream] " Primoz Fiser 2024-03-25 8:55 ` Primoz Fiser 2024-03-25 14:45 ` Jonathan Cameron 2024-03-25 14:45 ` Jonathan Cameron 2024-03-29 7:58 ` Primoz Fiser 2024-03-29 7:58 ` Primoz Fiser
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20240324135559.1640551d@jic23-huawei \ --to=jic23@kernel.org \ --cc=andrej.picej@norik.com \ --cc=conor+dt@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=haibo.chen@nxp.com \ --cc=imx@lists.linux.dev \ --cc=kernel@pengutronix.de \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=lars@metafoo.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=robh@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=shawnguo@kernel.org \ --cc=upstream@lists.phytec.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.