Commit 97847240 authored by David Gow's avatar David Gow Committed by Shuah Khan
Browse files

mmc: sdhci-of-aspeed: Remove some unnecessary casts from KUnit tests 



With KUnit's EXPECT macros no longer typechecking arguments as strictly,
get rid of a number of now unnecessary casts.

Signed-off-by: default avatarDavid Gow <davidgow@google.com>
Acked-by: default avatarAndrew Jeffery <andrew@aj.id.au>
Acked-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Reviewed-by: default avatarBrendan Higgins <brendanhiggins@google.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent 28dce2c4

drivers/mmc/host/sdhci-of-aspeed-test.c

+17 −17
Original line number Diff line number Diff line
@@ -26,23 +26,23 @@ static void aspeed_sdhci_phase_ddr52(struct kunit *test) 
	KUNIT_EXPECT_EQ(test, 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 25));



	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 0,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 0,

			aspeed_sdhci_phase_to_tap(NULL, rate, 180));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 0,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 0,

			aspeed_sdhci_phase_to_tap(NULL, rate, 181));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

			aspeed_sdhci_phase_to_tap(NULL, rate, 182));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

			aspeed_sdhci_phase_to_tap(NULL, rate, 183));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 2,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 2,

			aspeed_sdhci_phase_to_tap(NULL, rate, 184));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 3,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 3,

			aspeed_sdhci_phase_to_tap(NULL, rate, 185));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 14,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 14,

			aspeed_sdhci_phase_to_tap(NULL, rate, 203));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 204));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 205));

}
@@ -67,21 +67,21 @@ static void aspeed_sdhci_phase_hs200(struct kunit *test) 
	KUNIT_EXPECT_EQ(test, 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 96));



	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK,

			aspeed_sdhci_phase_to_tap(NULL, rate, 180));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK,

			aspeed_sdhci_phase_to_tap(NULL, rate, 185));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

			aspeed_sdhci_phase_to_tap(NULL, rate, 186));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 1,

			aspeed_sdhci_phase_to_tap(NULL, rate, 187));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 14,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 14,

			aspeed_sdhci_phase_to_tap(NULL, rate, 269));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 270));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 271));

	KUNIT_EXPECT_EQ(test, (int)ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

	KUNIT_EXPECT_EQ(test, ASPEED_SDHCI_TAP_PARAM_INVERT_CLK | 15,

			aspeed_sdhci_phase_to_tap(NULL, rate, 276));

}