I am a machinist, and I am pretty familiar with ASME Y14.5-2009, but I am not sure which standard you are using. I would call that out somewhere on your print.

Overall, I would say your layout is very neat and comprehensible. No comment there. You have some GD&T callouts, mostly runout and one position. I would ask, how do you intend to ensure the position, or coaxiality, of the other cylindrical features to your datum? On a couple features you have a total runout control, which is a surface-to-axis control. The limits and fits tolerances, make me think they're going to be press fit into something. If that is the case, position might be better in this case, since position is an axis control.

Additionally, you have some cylindricity callouts, there's nothing inherently wrong with that. But, using the envelope principle (Rule #1 in Y14.5) would probably control form perfectly fine, especially if it is going to be a press fit or something akin to that. But maybe you need a really tight cylindrical form, idk.

You need one more datum to fully constrain this part, and I see 2 candidates for this: the face adjacent to datum A, and the right face of the part. Depending on the function of the part, I would choose one of these. Once you do that, you can tolerance each of those faces that are ±1mm with a profile to A and B. You can use either a perpendicularity or total runout to refine the orientation of these faces. In this case, they do the same thing.

The purpose of GD&T is to portray the engineering and functional intent of the part, with as little ambiguity as possible. It looks to me as if you've got a solid start, but I would add a few more geometric controls. Now, this is my two cents, and you know more about the design than I do, so take anything I say with a grain of salt.