I mean ALL of the below to be helpful, I am not being rude or mean, but truth is a lot easier to write than lying, and lying doesn't help you at all. 
What I see is an average attempt of a kind'a boring logo... heheh
Let me explain, I 'assumed' you were a newbie by the logo attempt, so I looked at your website and I was pleasantly surprised that you have some really good shots, and you aren't a newbie. 
I then came back to look at the logo again, because really, through the years, I see all kinds of people that think that because they have a camera, they think they are a "shutterpro", but this gave it new light to me.
(I mean this in a joking, funny way) 
First, please get rid of the background, it is not helping things at all, it's way too busy, and it's really, really distracting.
Make a logo for a dark background, and another for a light background if you need to, but don't place a permanent one in it if you can help it, because sometimes you need to place it on shots that will clash with you logo's background.
(You can make a .gif that allows you to do the above, and it's a small file size too, it works pretty well)
Change the font to a more plain one, preferably without serifs, serifs and wilder fonts also distract, make things more busy, and don't do a think to help you out.
The curvature of the name is fine, if you like that, but circular would be best, I feel anyway, but if you like it, it'll work.
The aperture that you made appears to be either semi-transparent, or it also has part of the background in it, but it needs to be 'at least' 100% solid black.
Personally, I'd make the entire logo the same color, white, then you can ghost it, or whatever, plus, it's more professional that way.
You can also make it black for lighter backgrounds too, that is an acceptable way to do that.
If you plan on having a logo for you website only, make it solid, but if you also plan on using it as your signature on all of your shots, then it needs to be ghosted back, that way it's not harsh on our vision when we look at your work, or, if you have a client that is looking at it.
Most darkrooms in the past used a medium gray to make sure the light didn't mess with any of the photography process, in a way, anything (usually) other than gray does mess with our eyes, some.
If you look at a lot of photography websites you'll see dark grays as their backgrounds, that is done to make sure you are not distracted when viewing their works.
'Any' distractions is one too many, so the best you can do is to try and make sure you get rid of as many as you can.
People will divert from this of course, but if you can, make it one color, and as uncomplicated as possible, you'll make it easier on your customers/viewers that way.
Another thing is, if you printed this on your business card with the background, it'd be a rectangular box on a (I assume) a white background, not too professional looking, and really kind'a messy.
These are just a few things that will (hopefully) help you make a better logo, I don't know what imaging program you're using, but I'm just assuming Photoshop, you really need something with Layer capabilities really to make this good.
If you need further help, I'd be glad to kind'a walk you through a process or two, just PM me if you want, I don't mind, I just don't like to go through a process online with a lot of people's inputs, that can get crazy in a hurry. heheh
I have a lot of logos that I have made online that I can show you too.
I could also send you a .psd file if you wanted, that can clear up a lot in a hurry.
Anyway, good luck on this, however you go about it.
Randy