Re: [PATCH] ata: pata-imx: simplify assignment of device name

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

 



On Mon, Jan 09, 2017 at 09:53:36AM +0100, Sascha Hauer wrote:
> On Fri, Dec 23, 2016 at 10:50:46AM +0100, Uwe Kleine-König wrote:
> > Hello,
> > 
> > On Fri, Dec 23, 2016 at 10:45:49AM +0100, Uwe Kleine-König wrote:
> > >  	dev->detect = pata_imx_detect;
> > 
> > BTW, another thing I noticed while reading through this driver is:
> > 
> > pata_imx_detect is a wrapper around ata_port_detect. Later in
> > imx_pata_probe the function ide_port_register is called which also calls
> > ata_port_detect.
> > 
> > I don't have a machine handy where I could test, but I guess
> > pata_imx_detect could be dropped. (Or alternatively ide_port_register
> > could be reworked and all callers fixed to setup a detect call.)
> 
> 
> ide_port_register() has:
> 
> 	ret = ata_port_register(&ide->port);
> 	if (!ret)
> 		ata_port_detect(&ide->port);
> 
> The call to ata_port_detect() goes back to:
> 
> commit 9c8f73ba86884026f4bbfc6f593d73741cfebcdc
> Author: Michel Stam <m.stam@xxxxxxxx>
> Date:   Mon Apr 7 12:01:21 2014 +0200
> 
> The hunk adding ata_port_detect() is wrong as it shortcircuits the
> optional detection mechanism.

Who fixes this? /me looks at Michel who was added to Cc: :-)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux