Wordpress H1 Question

Currently a Wordpress theme of mine has the header being H1 and I
don't want it to be. To change this, would this work? This shows up on all my posts it appears, so in part I believe it detracts from my SEO as I want it to focus on the title and so on, not to take in to consideration a picture?

In styles.css, changing the logo part from...

#header h1.logo{
float:left;
padding:25px;

to...

#header span.logo{
float:left;
padding:25px;

and header.php from...

<h1 class="<?php if(get_option('pr_logo')){ echo "logo"; }else{echo "logo_text";} ?>">

<a href="<?php echo home_url(); ?>"><?php if(!get_option('pr_logo')){ bloginfo('title');} if(get_option('pr_logo')){ ?><img src="<?php echo get_option('pr_logo'); ?>" alt="<?php bloginfo('name'); ?>" /><?php }?></a></h1>

to...

<span class="<?php if(get_option('pr_logo')){ echo "logo"; }else{echo "logo_text";} ?>">

<a href="<?php echo home_url(); ?>"><?php if(!get_option('pr_logo')){ bloginfo('title');} if(get_option('pr_logo')){ ?><img src="<?php echo get_option('pr_logo'); ?>" alt="<?php bloginfo('name'); ?>" /><?php }?></a></span>

-------

Is that how I would "un-H1 it"? Or is there a better way?

To see the theme live you can click one of several links in my sig: Dedicated Server, Email Marketing, or VPS.

Would it be best to do this -- to make the header not H1? Does it make any sense for it to be H1?

Thanks so much for all the help.