forked from mirrors/linux
		
	scsi: sd: Inline sd_probe_part2()
Make sd_probe() easier to read by inlining sd_probe_part2(). This patch does not change any functionality. Cc: Lee Duncan <lduncan@suse.com> Cc: Hannes Reinecke <hare@suse.com> Cc: Luis Chamberlain <mcgrof@kernel.org> Cc: Johannes Thumshirn <jthumshirn@suse.de> Cc: Christoph Hellwig <hch@lst.de> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Bart Van Assche <bvanassche@acm.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
		
							parent
							
								
									21e6ba3f0e
								
							
						
					
					
						commit
						d16ece577b
					
				
					 1 changed files with 43 additions and 58 deletions
				
			
		|  | @ -3287,63 +3287,6 @@ static int sd_format_disk_name(char *prefix, int index, char *buf, int buflen) | |||
| 	return 0; | ||||
| } | ||||
| 
 | ||||
| static void sd_probe_part2(struct scsi_disk *sdkp) | ||||
| { | ||||
| 	struct scsi_device *sdp; | ||||
| 	struct gendisk *gd; | ||||
| 	u32 index; | ||||
| 	struct device *dev; | ||||
| 
 | ||||
| 	sdp = sdkp->device; | ||||
| 	gd = sdkp->disk; | ||||
| 	index = sdkp->index; | ||||
| 	dev = &sdp->sdev_gendev; | ||||
| 
 | ||||
| 	gd->major = sd_major((index & 0xf0) >> 4); | ||||
| 	gd->first_minor = ((index & 0xf) << 4) | (index & 0xfff00); | ||||
| 
 | ||||
| 	gd->fops = &sd_fops; | ||||
| 	gd->private_data = &sdkp->driver; | ||||
| 	gd->queue = sdkp->device->request_queue; | ||||
| 
 | ||||
| 	/* defaults, until the device tells us otherwise */ | ||||
| 	sdp->sector_size = 512; | ||||
| 	sdkp->capacity = 0; | ||||
| 	sdkp->media_present = 1; | ||||
| 	sdkp->write_prot = 0; | ||||
| 	sdkp->cache_override = 0; | ||||
| 	sdkp->WCE = 0; | ||||
| 	sdkp->RCD = 0; | ||||
| 	sdkp->ATO = 0; | ||||
| 	sdkp->first_scan = 1; | ||||
| 	sdkp->max_medium_access_timeouts = SD_MAX_MEDIUM_TIMEOUTS; | ||||
| 
 | ||||
| 	sd_revalidate_disk(gd); | ||||
| 
 | ||||
| 	gd->flags = GENHD_FL_EXT_DEVT; | ||||
| 	if (sdp->removable) { | ||||
| 		gd->flags |= GENHD_FL_REMOVABLE; | ||||
| 		gd->events |= DISK_EVENT_MEDIA_CHANGE; | ||||
| 	} | ||||
| 
 | ||||
| 	blk_pm_runtime_init(sdp->request_queue, dev); | ||||
| 	device_add_disk(dev, gd, NULL); | ||||
| 	if (sdkp->capacity) | ||||
| 		sd_dif_config_host(sdkp); | ||||
| 
 | ||||
| 	sd_revalidate_disk(gd); | ||||
| 
 | ||||
| 	if (sdkp->security) { | ||||
| 		sdkp->opal_dev = init_opal_dev(sdp, &sd_sec_submit); | ||||
| 		if (sdkp->opal_dev) | ||||
| 			sd_printk(KERN_NOTICE, sdkp, "supports TCG Opal\n"); | ||||
| 	} | ||||
| 
 | ||||
| 	sd_printk(KERN_NOTICE, sdkp, "Attached SCSI %sdisk\n", | ||||
| 		  sdp->removable ? "removable " : ""); | ||||
| 	scsi_autopm_put_device(sdp); | ||||
| } | ||||
| 
 | ||||
| /**
 | ||||
|  *	sd_probe - called during driver initialization and whenever a | ||||
|  *	new scsi device is attached to the system. It is called once | ||||
|  | @ -3433,7 +3376,49 @@ static int sd_probe(struct device *dev) | |||
| 	get_device(dev); | ||||
| 	dev_set_drvdata(dev, sdkp); | ||||
| 
 | ||||
| 	sd_probe_part2(sdkp); | ||||
| 	gd->major = sd_major((index & 0xf0) >> 4); | ||||
| 	gd->first_minor = ((index & 0xf) << 4) | (index & 0xfff00); | ||||
| 
 | ||||
| 	gd->fops = &sd_fops; | ||||
| 	gd->private_data = &sdkp->driver; | ||||
| 	gd->queue = sdkp->device->request_queue; | ||||
| 
 | ||||
| 	/* defaults, until the device tells us otherwise */ | ||||
| 	sdp->sector_size = 512; | ||||
| 	sdkp->capacity = 0; | ||||
| 	sdkp->media_present = 1; | ||||
| 	sdkp->write_prot = 0; | ||||
| 	sdkp->cache_override = 0; | ||||
| 	sdkp->WCE = 0; | ||||
| 	sdkp->RCD = 0; | ||||
| 	sdkp->ATO = 0; | ||||
| 	sdkp->first_scan = 1; | ||||
| 	sdkp->max_medium_access_timeouts = SD_MAX_MEDIUM_TIMEOUTS; | ||||
| 
 | ||||
| 	sd_revalidate_disk(gd); | ||||
| 
 | ||||
| 	gd->flags = GENHD_FL_EXT_DEVT; | ||||
| 	if (sdp->removable) { | ||||
| 		gd->flags |= GENHD_FL_REMOVABLE; | ||||
| 		gd->events |= DISK_EVENT_MEDIA_CHANGE; | ||||
| 	} | ||||
| 
 | ||||
| 	blk_pm_runtime_init(sdp->request_queue, dev); | ||||
| 	device_add_disk(dev, gd, NULL); | ||||
| 	if (sdkp->capacity) | ||||
| 		sd_dif_config_host(sdkp); | ||||
| 
 | ||||
| 	sd_revalidate_disk(gd); | ||||
| 
 | ||||
| 	if (sdkp->security) { | ||||
| 		sdkp->opal_dev = init_opal_dev(sdp, &sd_sec_submit); | ||||
| 		if (sdkp->opal_dev) | ||||
| 			sd_printk(KERN_NOTICE, sdkp, "supports TCG Opal\n"); | ||||
| 	} | ||||
| 
 | ||||
| 	sd_printk(KERN_NOTICE, sdkp, "Attached SCSI %sdisk\n", | ||||
| 		  sdp->removable ? "removable " : ""); | ||||
| 	scsi_autopm_put_device(sdp); | ||||
| 
 | ||||
| 	return 0; | ||||
| 
 | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Bart Van Assche
						Bart Van Assche