Re: [PATCH 2/2] drm/cmdline-parser: Use assert when needed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michał

On 8/4/22 10:17, Michał Winiarski wrote:
> Expecting to observe a specific value, when the function responsible for
> setting the value has failed will lead to extra noise in test output.
> Use assert when the situation calls for it.
> Also - very small tidying up around the changed areas (whitespace /
> variable locality).
> 
> Signed-off-by: Michał Winiarski <michal.winiarski@xxxxxxxxx>
> ---
>  .../gpu/drm/tests/drm_cmdline_parser_test.c   | 93 +++++++++----------
>  1 file changed, 46 insertions(+), 47 deletions(-)
> 
> diff --git a/drivers/gpu/drm/tests/drm_cmdline_parser_test.c b/drivers/gpu/drm/tests/drm_cmdline_parser_test.c
> index 058808faaf4a..7a313e2fd52a 100644
> --- a/drivers/gpu/drm/tests/drm_cmdline_parser_test.c
> +++ b/drivers/gpu/drm/tests/drm_cmdline_parser_test.c
> @@ -16,7 +16,7 @@ static void drm_cmdline_test_force_e_only(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "e";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -34,7 +34,7 @@ static void drm_cmdline_test_force_D_only_not_digital(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "D";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -47,16 +47,16 @@ static void drm_cmdline_test_force_D_only_not_digital(struct kunit *test)
>  	KUNIT_EXPECT_EQ(test, mode.force, DRM_FORCE_ON);
>  }
>  
> -static const struct drm_connector connector_hdmi = {
> -	.connector_type	= DRM_MODE_CONNECTOR_HDMIB,
> -};

Minor nit: I guess you miss to delete a line.

>  
>  static void drm_cmdline_test_force_D_only_hdmi(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> +	const struct drm_connector connector_hdmi = {
> +		.connector_type	= DRM_MODE_CONNECTOR_HDMIB,
> +	};


As the kernel test robot pointed out, this leads to stack issues (also
with PowerPC). You might want to leave it as it was or allocate it with
kunit_kzalloc.

Best Regards,
- Maíra Canal


>  	const char *cmdline = "D";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &connector_hdmi, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -69,16 +69,15 @@ static void drm_cmdline_test_force_D_only_hdmi(struct kunit *test)
>  	KUNIT_EXPECT_EQ(test, mode.force, DRM_FORCE_ON_DIGITAL);
>  }
>  
> -static const struct drm_connector connector_dvi = {
> -	.connector_type	= DRM_MODE_CONNECTOR_DVII,
> -};
> -
>  static void drm_cmdline_test_force_D_only_dvi(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> +	const struct drm_connector connector_dvi = {
> +		.connector_type = DRM_MODE_CONNECTOR_DVII,
> +	};>  	const char *cmdline = "D";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &connector_dvi, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -96,7 +95,7 @@ static void drm_cmdline_test_force_d_only(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "d";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -114,7 +113,7 @@ static void drm_cmdline_test_res(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -136,7 +135,7 @@ static void drm_cmdline_test_res_vesa(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480M";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -158,7 +157,7 @@ static void drm_cmdline_test_res_vesa_rblank(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480MR";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -180,7 +179,7 @@ static void drm_cmdline_test_res_rblank(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480R";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -202,7 +201,7 @@ static void drm_cmdline_test_res_bpp(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480-24";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -225,7 +224,7 @@ static void drm_cmdline_test_res_refresh(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480@60";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -248,7 +247,7 @@ static void drm_cmdline_test_res_bpp_refresh(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480-24@60";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -272,7 +271,7 @@ static void drm_cmdline_test_res_bpp_refresh_interlaced(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480-24@60i";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -294,9 +293,9 @@ static void drm_cmdline_test_res_bpp_refresh_interlaced(struct kunit *test)
>  static void drm_cmdline_test_res_bpp_refresh_margins(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> -	const char *cmdline =  "720x480-24@60m";
> +	const char *cmdline = "720x480-24@60m";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -318,9 +317,9 @@ static void drm_cmdline_test_res_bpp_refresh_margins(struct kunit *test)
>  static void drm_cmdline_test_res_bpp_refresh_force_off(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> -	const char *cmdline =  "720x480-24@60d";
> +	const char *cmdline = "720x480-24@60d";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -342,9 +341,9 @@ static void drm_cmdline_test_res_bpp_refresh_force_off(struct kunit *test)
>  static void drm_cmdline_test_res_bpp_refresh_force_on(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> -	const char *cmdline =  "720x480-24@60e";
> +	const char *cmdline = "720x480-24@60e";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -368,7 +367,7 @@ static void drm_cmdline_test_res_bpp_refresh_force_on_analog(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480-24@60D";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -390,12 +389,12 @@ static void drm_cmdline_test_res_bpp_refresh_force_on_analog(struct kunit *test)
>  static void drm_cmdline_test_res_bpp_refresh_force_on_digital(struct kunit *test)
>  {
>  	struct drm_cmdline_mode mode = { };
> -	static const struct drm_connector connector = {
> +	const struct drm_connector connector = {
>  		.connector_type = DRM_MODE_CONNECTOR_DVII,
>  	};
>  	const char *cmdline = "720x480-24@60D";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -443,7 +442,7 @@ static void drm_cmdline_test_res_margins_force_on(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480me";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -465,7 +464,7 @@ static void drm_cmdline_test_res_vesa_margins(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480Mm";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -487,7 +486,7 @@ static void drm_cmdline_test_name(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "NTSC";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_STREQ(test, mode.name, "NTSC");
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -499,7 +498,7 @@ static void drm_cmdline_test_name_bpp(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "NTSC-24";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_STREQ(test, mode.name, "NTSC");
>  
> @@ -514,7 +513,7 @@ static void drm_cmdline_test_name_option(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "NTSC,rotate=180";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_STREQ(test, mode.name, "NTSC");
> @@ -526,7 +525,7 @@ static void drm_cmdline_test_name_bpp_option(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "NTSC-24,rotate=180";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_STREQ(test, mode.name, "NTSC");
> @@ -540,7 +539,7 @@ static void drm_cmdline_test_rotate_0(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,rotate=0";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -563,7 +562,7 @@ static void drm_cmdline_test_rotate_90(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,rotate=90";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -586,7 +585,7 @@ static void drm_cmdline_test_rotate_180(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,rotate=180";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -609,7 +608,7 @@ static void drm_cmdline_test_rotate_270(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,rotate=270";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -632,7 +631,7 @@ static void drm_cmdline_test_hmirror(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,reflect_x";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -655,7 +654,7 @@ static void drm_cmdline_test_vmirror(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,reflect_y";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -679,7 +678,7 @@ static void drm_cmdline_test_margin_options(struct kunit *test)
>  	const char *cmdline =
>  		"720x480,margin_right=14,margin_left=24,margin_bottom=36,margin_top=42";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -705,7 +704,7 @@ static void drm_cmdline_test_multiple_options(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480,rotate=270,reflect_x";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -728,7 +727,7 @@ static void drm_cmdline_test_bpp_extra_and_option(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480-24e,rotate=180";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -752,7 +751,7 @@ static void drm_cmdline_test_extra_and_option(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "720x480e,rotate=180";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_TRUE(test, mode.specified);
>  	KUNIT_EXPECT_EQ(test, mode.xres, 720);
> @@ -774,7 +773,7 @@ static void drm_cmdline_test_freestanding_options(struct kunit *test)
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "margin_right=14,margin_left=24,margin_bottom=36,margin_top=42";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);
> @@ -797,7 +796,7 @@ static void drm_cmdline_test_freestanding_force_e_and_options(struct kunit *test
>  	struct drm_cmdline_mode mode = { };
>  	const char *cmdline = "e,margin_right=14,margin_left=24,margin_bottom=36,margin_top=42";
>  
> -	KUNIT_EXPECT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
> +	KUNIT_ASSERT_TRUE(test, drm_mode_parse_command_line_for_connector(cmdline,
>  									  &no_connector, &mode));
>  	KUNIT_EXPECT_FALSE(test, mode.specified);
>  	KUNIT_EXPECT_FALSE(test, mode.refresh_specified);



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux