Skip to contentSkip to conversation

New NAND driver

DISCUSSION Lee Jones 3 participants 6 messages Mar 24, 2014 – Mar 25, 2014 Memory technology devices (MTD)

lore
Share & tools
lore

Lee Jones

author
#0 lore raw

Hi Brian,

Firstly I'd like to thank you for supporting the inclusion of ST's SPI NOR (FSM) driver. As promised I will convert it over to the new framework once it has been applied by your good self.

In the meantime however, I have a NAND driver which I need to submit. I wanted to give you a heads-up prior to sending it in case you had any special requests. FYI: Shoving it where the sun don't shine, is not a valid request by the way :). It's 2700 lines and I've split it over 47 patches for reviewing purposes.

Sorry to do this to you Brian, I am sensitive to how over-burdened you are with the subsystem already. Perhaps it might make sense to put out a plea for volunteers to help with maintain MTD subsystem on LKML?

Anyway, what's the best thing to do in this instance? Should I just send it to the list as an RFC and be done with it?

Kind regards,
Lee

Signature · 4 lines
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
#1 lore raw
Quoted 11 lines · Lee Jones

Hi Brian,

Firstly I'd like to thank you for supporting the inclusion of ST's SPI NOR (FSM) driver. As promised I will convert it over to the new framework once it has been applied by your good self.

In the meantime however, I have a NAND driver which I need to submit. I wanted to give you a heads-up prior to sending it in case you had any special requests. FYI: Shoving it where the sun don't shine, is not a valid request by the way :). It's 2700 lines and I've split it over 47 patches for reviewing purposes.

One nand driver in 47 patches ? Whoa ;-)

Best regards,
Marek Vasut

#2 lore raw
In reply to Lee Jones

Hi,

Quoted 8 lines

From: Lee Jones

Hi Brian,

Firstly I'd like to thank you for supporting the inclusion of ST's SPI NOR (FSM) driver. As promised I will convert it over to the new framework once it has been applied by your good self.

Quoted 6 lines

In the meantime however, I have a NAND driver which I need to submit. I wanted to give you a heads-up prior to sending it in case you had any special requests. FYI: Shoving it where the sun don't shine, is not a valid request by the way :). It's 2700 lines and I've split it over 47 patches for reviewing purposes.

Will it be possible for you to send the patches split in one patch per file ? As switching back and forth, between 47 patches may be bit troublesome. Later you may split it as you or others like..

Quoted 4 lines

Sorry to do this to you Brian, I am sensitive to how over-burdened you are with the subsystem already. Perhaps it might make sense to put out a plea for volunteers to help with maintain MTD subsystem on LKML?

I can help reviewing those patches, if you can please share your controller specs, and also share if there are any already know hardware limitations.

Quoted 3 lines

Anyway, what's the best thing to do in this instance? Should I just send it to the list as an RFC and be done with it?

with regards, pekon

author
#3 lore raw

Morning Pekon,

Quoted 14 lines
Quoted 8 lines

From: Lee Jones

Hi Brian,

Firstly I'd like to thank you for supporting the inclusion of ST's SPI NOR (FSM) driver. As promised I will convert it over to the new framework once it has been applied by your good self.

Quoted 6 lines

In the meantime however, I have a NAND driver which I need to submit. I wanted to give you a heads-up prior to sending it in case you had any special requests. FYI: Shoving it where the sun don't shine, is not a valid request by the way :). It's 2700 lines and I've split it over 47 patches for reviewing purposes.

Quoted 3 lines

Will it be possible for you to send the patches split in one patch per file ? As switching back and forth, between 47 patches may be bit troublesome. Later you may split it as you or others like..

There is only one file. I can send it as a single patch; however, when I sent the NOR driver [1], it was considered helpful to review in chunks, kinda how the driver was written. I'll send it as a set to start with and if you think it creates more problems than it solves then we'll do something about it.

[1] https://lkml.org/lkml/2014/3/20/103

Quoted 7 lines
Quoted 4 lines

Sorry to do this to you Brian, I am sensitive to how over-burdened you are with the subsystem already. Perhaps it might make sense to put out a plea for volunteers to help with maintain MTD subsystem on LKML?

I can help reviewing those patches, if you can please share your controller specs, and also share if there are any already know hardware limitations.

Thanks for your kind offer. I will send it right away.

Quoted 2 lines
Quoted 2 lines

Anyway, what's the best thing to do in this instance? Should I just send it to the list as an RFC and be done with it?

Kind regards,
Lee

Signature · 4 lines
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
#4 lore raw

Hello Lee,

[...]

Quoted 6 lines
Quoted 3 lines

I can help reviewing those patches, if you can please share your controller specs, and also share if there are any already know hardware limitations.

Thanks for your kind offer. I will send it right away.

Is the spec for this controller available in public domain somewhere ?

with regards, pekon

author
#5 lore raw

Hi Pekon,

Quoted 8 lines

[...]

Quoted 6 lines
Quoted 3 lines

I can help reviewing those patches, if you can please share your controller specs, and also share if there are any already know hardware limitations.

Thanks for your kind offer. I will send it right away.

Is the spec for this controller available in public domain somewhere ?

I don't think it is. To my knowledge all of the public NAND documentation can be found here:

http://www.stlinux.com/howto/NAND

I appreciate that this can make things a little awkward, but these decisions are made well above my pay grade.

Kind regards,
Lee

Signature · 4 lines
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog