Re: [PATCH v2 01/15] spi: Remove unneeded OF node NULL checks

[PATCH v2 00/15] spi: Header and core clean up and refactoring · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 02/15] spi: Drop duplicate IDR allocation code in spi_register_controller() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 02/15] spi: Drop duplicate IDR allocation code in spi_register_controller() · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 02/15] spi: Drop duplicate IDR allocation code in spi_register_controller() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Mark Brown <broonie@kernel.org> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Mark Brown <broonie@kernel.org> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · Mark Brown <broonie@kernel.org> · 2023-07-11
Re: [PATCH v2 04/15] spi: Replace open coded spi_controller_xfer_timeout() · AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> · 2023-07-11
[PATCH v2 01/15] spi: Remove unneeded OF node NULL checks · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 01/15] spi: Remove unneeded OF node NULL checks · AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> · 2023-07-11
Re: [PATCH v2 01/15] spi: Remove unneeded OF node NULL checks · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 05/15] spi: Remove code duplication in spi_add_device_locked() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 05/15] spi: Remove code duplication in spi_add_device_locked() · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 05/15] spi: Remove code duplication in spi_add_device_locked() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: [PATCH v2 05/15] spi: Remove code duplication in spi_add_device_locked() · Sebastian Reichel <hidden> · 2023-07-11
Re: [PATCH v2 05/15] spi: Remove code duplication in spi_add_device_locked() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 03/15] spi: Replace if-else-if by bitops and multiplications · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 03/15] spi: Replace if-else-if by bitops and multiplications · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 03/15] spi: Replace if-else-if by bitops and multiplications · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 08/15] spi: Clean up headers · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 08/15] spi: Clean up headers · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 08/15] spi: Clean up headers · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: [PATCH v2 08/15] spi: Clean up headers · Mark Brown <broonie@kernel.org> · 2023-07-11
Re: [PATCH v2 08/15] spi: Clean up headers · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 06/15] spi: Use sysfs_emit() to instead of s*printf() · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 07/15] spi: Sort headers alphabetically · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 09/15] spi: Use struct_size() helper · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 09/15] spi: Use struct_size() helper · Marc Kleine-Budde <mkl@pengutronix.de> · 2023-07-10
Re: [PATCH v2 09/15] spi: Use struct_size() helper · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 09/15] spi: Use struct_size() helper · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 11/15] spi: Get rid of old SPI_MASTER_NO_TX & SPI_MASTER_NO_RX · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 10/15] spi: Use predefined constants from bits.h and units.h · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 14/15] spi: Convert to SPI_CONTROLLER_HALF_DUPLEX · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 12/15] spi: Get rid of old SPI_MASTER_MUST_TX & SPI_MASTER_MUST_RX · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
[PATCH v2 13/15] spi: Rename SPI_MASTER_GPIO_SS to SPI_CONTROLLER_GPIO_SS · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 13/15] spi: Rename SPI_MASTER_GPIO_SS to SPI_CONTROLLER_GPIO_SS · Serge Semin <hidden> · 2023-07-11
Re: [PATCH v2 13/15] spi: Rename SPI_MASTER_GPIO_SS to SPI_CONTROLLER_GPIO_SS · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
[PATCH v2 15/15] spi: Fix spelling typos and acronyms capitalization · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-10
Re: [PATCH v2 00/15] spi: Header and core clean up and refactoring · Mark Brown <broonie@kernel.org> · 2023-07-10
Re: [PATCH v2 00/15] spi: Header and core clean up and refactoring · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: [PATCH v2 00/15] spi: Header and core clean up and refactoring · Mark Brown <broonie@kernel.org> · 2023-07-11
Re: [PATCH v2 00/15] spi: Header and core clean up and refactoring · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2023-07-11
Re: (subset) [PATCH v2 00/15] spi: Header and core clean up and refactoring · Mark Brown <broonie@kernel.org> · 2023-07-12

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2023-07-11 12:52:15
Also in: alsa-devel, linux-amlogic, linux-arm-msm, linux-mediatek, linux-riscv, linux-rockchip, linux-spi, linux-trace-kernel, lkml

On Tue, Jul 11, 2023 at 10:12:55AM +0200, AngeloGioacchino Del Regno wrote:

Il 10/07/23 17:49, Andy Shevchenko ha scritto:

quoted

In the couple of places the NULL check of OF node is implied by the call
that takes it as a parameter. Drop the respective duplicate checks.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Validated against spi-mt65xx, spi-mt7621, spi-mtk-nor, spi-mtk-snfi;

Reviewed-by: AngeloGioacchino Del Regno
[off-list ref] # MediaTek

By some reason the tag is split and I'm not sure `b4` can cope with that.
In any case, added manually. Thank you for the review!

-- 
With Best Regards,
Andy Shevchenko

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help