spi: spi-qpic-snand: remove superfluous parameters of qcom_spi_check_error()
authorGabor Juhos <j4g8y7@gmail.com>
Tue, 20 May 2025 19:08:56 +0000 (21:08 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 21 May 2025 17:56:08 +0000 (18:56 +0100)
commit0f529570ecaf99244dc86b8af13618f0d07b0e44
treef5fdda77efeb0134d3cee31f99da2a850ff50cdc
parent477d16c0919e82a7f5f673a6e3ae39a17c773037
spi: spi-qpic-snand: remove superfluous parameters of qcom_spi_check_error()

The qcom_spi_check_error() function determines the errors of a previous
page read operation solely by using the cached register values in the
register read buffer. The data pointed by the 'data_buf' and the 'oob_buf'
parameters are not used for that at all.

Remove the superfluous parameters of the function along with the related
local variables to simplify the code. Also, remove the variables from the
caller functions which became unused due to the change.

Note:
Althought the similar parse_read_errors() function in the 'qcom_nand'
driver has the same parameters, but that function passes down the
pointers to check_for_erased_page() at the end of the function.

It is not clear, that a similar call is missing here, or the superfluous
parameters are simply leftovers of the development process. Nevertheless,
if additional code is needed, the parameters can be added back later.

Signed-off-by: Gabor Juhos <j4g8y7@gmail.com>
Link: https://patch.msgid.link/20250520-qpic-snand-superfluous-params-v1-1-86dd4963e90f@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-qpic-snand.c