* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE [not found] <E1CdAeF-0007RX-Fq@phoenix.infradead.org> @ 2004-12-11 18:05 ` David Woodhouse 2004-12-11 18:21 ` Artem B. Bityuckiy 0 siblings, 1 reply; 3+ messages in thread From: David Woodhouse @ 2004-12-11 18:05 UTC (permalink / raw) To: Artem Bityuckiy; +Cc: linux-mtd On Sat, 2004-12-11 at 17:03 +0000, Artem Bityuckiy wrote: > Remove headers frome here. Instead join them and use one <linux/jffs3.h> > header. No. Keep it in the fs/jffs3 directory not in include/linux/ -- dwmw2 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE 2004-12-11 18:05 ` mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE David Woodhouse @ 2004-12-11 18:21 ` Artem B. Bityuckiy 2004-12-11 21:31 ` David Woodhouse 0 siblings, 1 reply; 3+ messages in thread From: Artem B. Bityuckiy @ 2004-12-11 18:21 UTC (permalink / raw) To: David Woodhouse; +Cc: MTD List On Sat, 11 Dec 2004, David Woodhouse wrote: > On Sat, 2004-12-11 at 17:03 +0000, Artem Bityuckiy wrote: > > Remove headers frome here. Instead join them and use one <linux/jffs3.h> > > header. > > No. Keep it in the fs/jffs3 directory not in include/linux/ > > -- > dwmw2 > Ok, but just interesting why? :-) You may not commit it to BK. patchin.sh doesn't install it unles it was explicitly asked... -- Best Regards, Artem B. Bityuckiy, St.-Petersburg, Russia. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE 2004-12-11 18:21 ` Artem B. Bityuckiy @ 2004-12-11 21:31 ` David Woodhouse 0 siblings, 0 replies; 3+ messages in thread From: David Woodhouse @ 2004-12-11 21:31 UTC (permalink / raw) To: Artem B. Bityuckiy; +Cc: MTD List On Sat, 11 Dec 2004, Artem B. Bityuckiy wrote: > > No. Keep it in the fs/jffs3 directory not in include/linux/ > > > Ok, but just interesting why? It's private to the file system. It doesn't need to live in include/linux where other people can see it. The only reason we _used_ to put them there is because of the inode and superblock unions. There _was_ a reason I moved it out of include/linux/ in the first place. -- dwmw2 ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-12-11 21:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <E1CdAeF-0007RX-Fq@phoenix.infradead.org>
2004-12-11 18:05 ` mtd/fs/jffs3 jffs3.h, 3.1, NONE jffs3_fs_i.h, 3.1, NONE jffs3_fs_sb.h, 3.1, NONE David Woodhouse
2004-12-11 18:21 ` Artem B. Bityuckiy
2004-12-11 21:31 ` David Woodhouse
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox